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
fix(client): improve string validation when trying to upload a dataset file by URL #2834
Conversation
You can access the deployment of this PR at https://renku-ci-ui-2834.dev.renku.ch |
@andre-code Is this PR waiting for a renku-core fix? |
I think the renku-core requirement is not a problem anymore (renku-core 2.8.0 is out) |
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.
This works fine now 👍
We can use the new function instead of the older isUrl
; that's actually used only in a single other place where the new function should be more reliable. I suggest replacing the import with isValidURL
and removing the older function isUrl
(changing the code is also fine; I find the new name you gave to the new function more intuitive)
787a3b1
to
eb3d218
Compare
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.
👍
Tearing down the temporary RenkuLab deplyoment for this PR. |
PR to fix bug when upload a dataset file by url.
Changes:
Fix #2801
/deploy extra-values=core.sentry.dsn=https://0cc02f3a02d3474d9f7e90915873ab4d@sentry.dev.renku.ch/2,core.sentry.environment=renku-dev,core.sentry.enabled=true