-
Notifications
You must be signed in to change notification settings - Fork 0
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
Share button #76
base: main
Are you sure you want to change the base?
Share button #76
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.
Looks good so far! I left a few questions and comments.
const addCaseUrl = Linking.createURL('?event=addCase&', { | ||
queryParams: { caseUid }, | ||
}); |
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.
event=addCase
is technically a query parameter, so maybe we can clean up this function instead of concatenating multiple partial strings.
src/app/(BottomTabNavigation)/AllCases/CaseScreen/[caseUid].tsx
Outdated
Show resolved
Hide resolved
await Share.share({ | ||
//message: 'ADD CASE', | ||
url: `${addCaseUrl}/`, |
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.
Out of curiosity, is there anyway to customize how the link looks when shared with someone? Sometimes apps will attach an image or additional text to the link so users don't have to see the entire link.
What's new in this PR
Description
Screenshots
How to review
Next steps
Relevant Links
Online sources
Related PRs
Deep Linking Base
CC: @ronniebeggs