-
Notifications
You must be signed in to change notification settings - Fork 986
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
Implement new shorter universal links #10083
Comments
@hesterbruikman can we include this for |
I think that's fine, as long as we have it by the end of this month. As I understand it, this isn't hugely complex or would risk any delays. Added v1.2 release label. Does this issue require any updates in https://github.com/status-im/universal-links-handler/? |
The necessary changes have already been made in status-im/universal-links-handler#29 and deployed. |
Then I'm confused 🙂 Are you saying it's not in the release cut for V1.1? |
It's very simple. There are two parts to this:
In order to change the structure of links under https://join.status.im/ two changes have to be made:
On a specific example. If you go to https://join.status.im/@jakubgs.eth - which is the new form of https://join.status.im/user/jakubgs.eth - it will display a page, but it will not work with the App, regardless of from where you open it. For that to work changes have to be made to |
Thanks for explaining!
Agree! I hate it when the short form doesn't work and I also get zero information about what it's trying to do |
Btw @jakubgs do you know why exactly we don't allow upper case, spaces, etc? I see a lot of value in being able to create a channel anywhere, just by editing a url. The format restrictions limit that feature. Nothing concrete yet, but thinking of how we can offer a 'create and share channel' feature on status.im |
I don't know exactly why, but it makes sense to me. Look at Twitter, hashtags can't have spaces or special characters either. I think it's sensible. And you still can "create" a channel by just writing down a URL like https://join.status.im/my-new-channel just that if you for example use uppercase it will not work: https://join.status.im/My-New-Channel What we could possibly do is simply redirect the upper case URL to the lower case URL. |
That would solve it. What I noticed before was the issue of having #friendschat when using Join Chat in Status (validation permits uppercase) and #Friendschat when using the url https://join.status.im/Friendschat. They'd open 2 different channels. |
There you go: status-im/universal-links-handler#32 |
#5302 is old but still relevant. We should fix the deep linking of upper case channel names. |
Due to issues found in #10222 I had to replace Fuck Apple. |
Feature Issue
The current form of links is too verbose and long. After discussing this with @jarradh and some others we've decided to use shorter forms, like so:
These were implemented in status-im/universal-links-handler#29 and are already deployed.
User Story
As a user, I want to shorter universal links so that it's easier to remember them and make them yourself. They also produce more readable QR codes.
Description
We will have to support both the old and the new way for at least one release to not break the old links. After that we can see in server logs how many people still use them and we can retire them eventually in a later release to avoid maintaining dead code.
Acceptance Criteria
New forms of universal links are supported on Android and iOS, in addition to old forms.
The text was updated successfully, but these errors were encountered: