-
Notifications
You must be signed in to change notification settings - Fork 156
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
can we clean up the URL path to avoid urlencoded /, eg #4595
Comments
might be related to spaces ... but there really is no reason to have |
This requires first a update to vue-router@4 which requires vue@3. @fschade thought of a dirty hack which could do this already now but we decided against it and delay this until the update of vue and vue-router. |
@LukasHirt any idea of the proposed update of the vue components? |
No progress so far. The update is not part of the current sprint. We were focusing on our accessibility tasks at the moment because of the WCAG certification. @labkode Is this ticket urgent for you? Last time I heard it wasn't, so we weren't putting too much priority to it. |
Hi @LukasHirt, it is as we plan to expose the new system, where user would be able to copy/paste the url to access the resources and if the URL changes this will break behaviour. |
@LukasHirt it can wait until next one, thanks for following it up |
At least the %2F encoding should be fixed via #5715 (patched until we switch to Vue3) and just landed in master, closing this issue for now (yet we'll perhaps need some more routing-related ones in the future) |
Files app use non-url encoded paths, just missing some media urls @kulmann knows. |
https://phoenix.owncloud.com/index.html#files/list/eos%2Fusers%2Fhugo
It would allow cnp the path and in the future make the url more similar when using other apps, eg for collabora:
https://phoenix.owncloud.com/index.html#collabora/eos%2Fusers%2Fhugo
?The text was updated successfully, but these errors were encountered: