-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: handle non blocking errors on fork #1351
Conversation
You can access the deployment of this PR at https://renku-ci-ui-1351.dev.renku.ch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this will solve UX issue and clarify what happened when a fork only partially succeeds. I added a commit to change the messages slightly.
I pushed a minor change to prevent linting errors |
Tearing down the temporary RenkuLab deplyoment for this PR. |
This improves the UX when a non-fatal error occurs while forking (i.e. the pipeline or webhook creation fails)
How to test: it's a bit tricky. The easiest way is running telepresence on the PR environment and manually modifying the code in the
client.forkProject
function to trigger the error on the target API (try with the main fork API, then withrunPipeline
andcreateGraphWebhook
)P.S. mind that the error message is slightly different than the suggested one, to keep it coherent with the other cases. Feel free to suggest changes if it's not clear (or change it directly in the PR).
/deploy renku=ui-design-test-fix
fix #1341