Skip to content
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

ft: Enable deep-links #343

Open
wants to merge 9 commits into
base: development
Choose a base branch
from

Conversation

wizlif
Copy link
Collaborator

@wizlif wizlif commented Jan 17, 2023

Enables ability to use deep-links to traverse the app routes.

Android

iOs

Issue: iOs launches safari with link for https schemes

Android Preview

Screen.Recording.2023-01-17.at.09.08.50.mov

iOs Preview

Screen.Recording.2023-01-17.at.09.23.58.mov
Screen.Recording.2023-01-17.at.09.29.00.mov

Closes #287

@Xazin
Copy link
Member

Xazin commented Jan 17, 2023

Don't we also need to set this up on our domains:
https://developer.apple.com/documentation/xcode/supporting-associated-domains

@codecov
Copy link

codecov bot commented Jan 17, 2023

Codecov Report

Merging #343 (25f89ae) into development (023d67e) will increase coverage by 2.22%.
The diff coverage is 60.00%.

@@               Coverage Diff               @@
##           development     #343      +/-   ##
===============================================
+ Coverage        66.55%   68.78%   +2.22%     
===============================================
  Files              157      157              
  Lines             4010     4017       +7     
===============================================
+ Hits              2669     2763      +94     
+ Misses            1341     1254      -87     
Impacted Files Coverage Δ
...rowdaction_details/crowdaction_details_screen.dart 58.86% <50.00%> (+58.86%) ⬆️
lib/presentation/home/widgets/password_modal.dart 92.85% <66.66%> (-2.67%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@wizlif
Copy link
Collaborator Author

wizlif commented Jan 17, 2023

@wizlif
Copy link
Collaborator Author

wizlif commented Jan 20, 2023

Screen.Recording.2023-01-20.at.19.44.21.mov

@wizlif wizlif requested review from Xazin, a team and phalcorine and removed request for a team January 20, 2023 16:53
@wizlif
Copy link
Collaborator Author

wizlif commented Jan 28, 2023

Simulator.Screen.Recording.-.iPhone.13.-.2023-01-28.at.21.19.35.mp4

Xazin
Xazin previously approved these changes Feb 23, 2023
run: |
echo "$DEEPLINK_XCCONFIG_CONTENTS" > ios/Flutter/Deeplink.xcconfig
env:
DEEPLINK_XCCONFIG_CONTENTS: ${{ secrets.IOS_DEEPLINK_CONFIG }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do we input here? Different ones depending on Development/Production? DM me on Slack

Copy link
Member

@Xazin Xazin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if it's rebased properly since I did some changes on development, but shouldn't be any destructive.

@Xazin
Copy link
Member

Xazin commented Jul 4, 2023

We need to rebase this, and then we should be able to merge.

@wizlif
Copy link
Collaborator Author

wizlif commented Jul 5, 2023

Let me cherry pick the major changes

@wizlif
Copy link
Collaborator Author

wizlif commented Jul 7, 2023

@Xazin Let’s first merge #343 before i rebase. Some minor conflicts between the two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[SUGGESTION]: Setup deeplinking
2 participants