Skip to content

feat(supabase_flutter): Update app_links dependency to v6.0.1 #915

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

Closed
wants to merge 1 commit into from

Conversation

flowhorn
Copy link

@flowhorn flowhorn commented May 6, 2024

What kind of change does this PR introduce?

With app_links version 6.0.0 there are breaking changes in the app_links api (https://pub.dev/packages/app_links/changelog):

Breaking: Renamed getInitialApp* and getLatestApp* methods to getInitialLink* and getLatestLink*.

I adjusted this in the PR and bumped the app_links package to ^6.0.1

Additional context

I don't know whether you directly want to bump the app_links version, as there were multiple breaking version releases in the last 2 months. Just wanted to make this small PR as I personally bumped the version for myself.

@dshukertjr
Copy link
Member

Yeah, this would be a breaking change on supabase_flutter for anyone using it with app_links, so we would wait until the next breaking change to introduce this update.

Maybe we can explore handling deep links without app_links dependencies? https://docs.flutter.dev/ui/navigation/deep-linking

@Vinzent03
Copy link
Collaborator

Can't we try to call the old methods with a try catch, and fallback to the new one? Definitely isn't the best way, but it should work.

@dshukertjr
Copy link
Member

@Vinzent03 I really don't think it's possible, but you have surprised me so many times in the past. Could you show me how that would work?

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.

3 participants