Psp 9512 jobs for upload, polling, retries #4529
Conversation
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
1 similar comment
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
devinleighsmith
force-pushed
the
psp-9512
branch
from
759e762 to
74bfee3
Compare
…recent copy of documentQueue -> since polling may trigger updates.
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
| /// </summary> | ||
| /// <returns></returns> | ||
| [HttpGet("{documentQueueId:long}")] | ||
| [HasPermission(Permissions.SystemAdmin)] |
There was a problem hiding this comment.
general question about permissions - I see all queue endpoints require sysadmin permissions. Is this intended? How will the frontend connect to this new API for non-admin users?
There was a problem hiding this comment.
the frontend won't connect to this api, at least, not directly. The documents will come in from the frontend via the document controller, which will handle adding the document queue entity. This controller that allows direct interaction with the document queue is really only used by the scheduler.
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
1 similar comment
|
See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/4529 |
No description provided.