Skip to content

feat(dynamic-links): forward errors to application#176

Merged
triniwiz merged 1 commit into
NativeScript:mainfrom
mukaschultze:feat/forward-dynamic-link-errors
Feb 9, 2023
Merged

feat(dynamic-links): forward errors to application#176
triniwiz merged 1 commit into
NativeScript:mainfrom
mukaschultze:feat/forward-dynamic-link-errors

Conversation

@mukaschultze

Copy link
Copy Markdown
Contributor

This PR adds an optional parameter to the onLink handler that indicates the error that occurred while fetching the target of a short dynamic link.

There are two reasons for this change:

  • This makes both iOS and Android behave the same for unknown links. Right now the Android handler emits null for unknown links, while iOS simply ignores them.
  • The error is useful for applications that need to show some sort of user message or dispatch an event upon malformed or unknown dynamic links.

BREAKING CHANGE: This change will call the dynamic link handler with null values on iOS. For Android, it'll be the same as before since the error param is optional.

@cla-bot cla-bot Bot added the cla: yes label Jan 24, 2023
@triniwiz triniwiz merged commit a50026e into NativeScript:main Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants