collaction_app icon indicating copy to clipboard operation
collaction_app copied to clipboard

ft: Enable deep-links

Open wizlif opened this issue 2 years ago • 8 comments

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

Android

  • [x] https://collaction.org
  • [x] app://collaction.org
  • [x] https://collaction.app
  • [x] app://collaction.app

iOs

  • [ ] https://collaction.org
  • [x] app://collaction.org
  • [ ] https://collaction.app
  • [x] app://collaction.app

Issue: iOs launches safari with link for https schemes

Android Preview

https://user-images.githubusercontent.com/15793624/212828992-9bf0bde6-cbf0-4baf-ad5c-ce1e24a3f7ec.mov

iOs Preview

https://user-images.githubusercontent.com/15793624/212829554-80bbbfaa-907b-43d2-b219-4f81d89a1575.mov

https://user-images.githubusercontent.com/15793624/212829419-f802ac13-a039-4eee-bb99-a171e6a5cd3b.mov

Closes #287

wizlif avatar Jan 17 '23 06:01 wizlif

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

Xazin avatar Jan 17 '23 07:01 Xazin

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%) :arrow_up:
lib/presentation/home/widgets/password_modal.dart 92.85% <66.66%> (-2.67%) :arrow_down:

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

codecov[bot] avatar Jan 17 '23 07:01 codecov[bot]

https://developer.apple.com/documentation/xcode/supporting-associated-domains

Yes we do, we could pass it via CI

wizlif avatar Jan 17 '23 18:01 wizlif

https://user-images.githubusercontent.com/15793624/213756508-68fb9f30-7b7f-42c1-a218-e6ce507bd98e.mov

wizlif avatar Jan 20 '23 16:01 wizlif

https://user-images.githubusercontent.com/15793624/215290642-99d53af2-2537-41be-b9a4-0601b9b3c7d7.mp4

wizlif avatar Jan 28 '23 21:01 wizlif

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

Xazin avatar Jul 04 '23 19:07 Xazin

Let me cherry pick the major changes

wizlif avatar Jul 05 '23 02:07 wizlif

@Xazin Let’s first merge https://github.com/CollActionteam/collaction_app/pull/343 before i rebase. Some minor conflicts between the two.

wizlif avatar Jul 07 '23 04:07 wizlif