Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Potential fix for issue #63
I will be using @SpaceLaunchNow (user_id= 841696172259606528) as the example account. The code as is first tries the URI - if a package is not available to handle the twitter:// URI then it falls back to the web URL.
As is the URI works ONLY with the user_id - however the web url ONLY works with the screen name.
Relevant intent logic here.
Example A - URI doesn’t work but weblink does:
twitter://user?user_id= SpaceLaunchNow
https://twitter.com/SpaceLaunchNow
Example B - URI works but weblink doesnt:
twitter://user?user_id=841696172259606528
https://twitter.com/841696172259606528
The obvious solution is to use just the screen_name or just the user_id for the URI and fallback link. From what I can see the web url only accepts screen names so changing the URI to screen_name makes more sense.