search

ubccpsc310 / classy

Course management and automation.
MIT License
2 stars 3 forks source link

Provisioning errors not shown in the admin ui for project provisioning #107

Open rtholmes opened 8 months ago

rtholmes commented 8 months ago

e.g., when AdminController::planProvision encounters an exception, this is not shown in the Project Management page UI, despite the object coming back to the frontend with something like:

{
    "failure": {
        "message": "Unable to provision repos: CustomCourseController::computeNames( ... ) - multiple teams found: ..."
...
rtholmes commented 5 months ago

Natasha also noted that the deliverable import URL should be specified if it is being used for provisioning. An error should be shown or the URL should be required.

Reid - is it possible to force the user to add an Import URL when creating a deliverable (as is done with the Deliverable Id) OR could the error message be more informative (for example you might currently get something like: “"Unable to provision repo: provisionRepository( tutorial_atest-02 ) failed; failed to create repo; ERROR: provisionRepository( tutorial_atest-02 ) creating repo failed; ERROR: Cannot read properties of undefined (reading 'includes')"”).

natashamm commented 4 months ago

FWIW another user has ran into this issue