-
Notifications
You must be signed in to change notification settings - Fork 2.7k
custom URL scheme upper case letters not working #2309
Conversation
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.
Thanks @Hobby-Student for the pull request!
From the generated test server, it appears custom URL schemes with upper case letters don't still render a link
Also @hmhealey - wondering if it was intended that only lower case schemes work?
no problem. I try to help... but for now, I'm stuck in searching the root of the problem. Your test servers are very helpful! |
Definitely not a problem! Let us know if you have any questions along the way :) |
How often is the Test Server updated? Only if someone of the team triggers it or automated? |
Hey @Hobby-Student, The Spinmint test server is used to test a PR. For developing a feature please use a local dev environment. You can find an instruction for this on https://developers.mattermost.com/contribute/webapp/developer-setup/. If you have problems feel free as them on the Mattermost Community Server. |
did read the dev instructions. will try to get a dev machine asap. |
Hey @Hobby-Student, Did you manage to set up a dev environment? Is there anything I can help you with? |
I'm currently waiting for some RAM. Should arrive the day after tomorrow. |
Instead of having the admin specify an uppercase custom URL scheme, would it make sense to make all of the custom URL schemes case-insensitive? |
exactly what I'm trying to do. If you could point me to the right direction, where custom schemes are parsed...? Android app seems to work case insensitive. |
I think this line is the one that needs to be changed since it currently looks for the exact string in the array. Using a |
Thank you @Hobby-Student! @hmhealey apart from what's shown in the screenshot above, any additional tests that we should do? If so, I can ask our QA, but in general it appears to work as expected. |
I don't think there's anything else specific to test. These changes are mostly around making it so that the URL scheme can contain uppercase letters |
Sounds good. Thanks @Hobby-Student for this! I requested a second dev review and then this should be ready to merge :) |
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.
Thanks @Hobby-Student, looks good to me. Please rebase and fix conflicts so we can merge it.
done. Glad that I was able help 😃 |
Thank you for the contribution @Hobby-Student 👍 |
Summary
Custom URL Schemes
field enter a value like "file, DMS, dms"I did so and the upper case one is not working. We have a software which is exporting links as
inserting this link into a message is not working. Manually replace DMS to
is working as a clickable link.
I don't know if this is the only place, but I didn't find any other... If this is only rendered on client side, the windows desktop is also affected (have not tested other apps).
Webapp -> uppercase not working
windows desktop app -> uppercase not working
Android App -> uppercase partially working (DMS does, HTTP does not)
Ticket Link
none
Checklist