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

[expo-auth-session] Fix release channel support #7687

Merged

Conversation

lukmccall
Copy link
Contributor

Why

Resolves #7674.

How

Fixes AuthSession.getDefaultReturnUrl()

Test Plan

  • run this code in unpublished and published app:
React.useEffect(() => {
    const f = async () => {
      alert(AuthSession.getDefaultReturnUrl());
    };
    f();
  }, []);
  • run adb shell am start -d <result of AuthSession.getDefaultReturnUrl()> (for me: "exp://exp.host/@lukaszkosmaty/sandbox/--/expo-auth-session?release-channel=releases-test")

@brentvatne
Copy link
Member

@lukmccall - can you write a test to cover this case?

packages/expo-auth-session/CHANGELOG.md Outdated Show resolved Hide resolved
@lukmccall lukmccall force-pushed the @lukmccall/auth-session-fix-support-for-release-channels branch from 5d7d32a to 2b9788f Compare April 9, 2020 10:55
Copy link
Member

@tsapeta tsapeta left a comment

Choose a reason for hiding this comment

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

Approved, but would be great to write a test as Brent suggested 😉

@lukmccall lukmccall merged commit e539e37 into master Apr 9, 2020
@lukmccall lukmccall deleted the @lukmccall/auth-session-fix-support-for-release-channels branch April 9, 2020 13:29
brentvatne pushed a commit that referenced this pull request Apr 16, 2020
* [expo-auth-session] Fix release channel support

* [expo-auth-session] Update changelog

* [expo-auth-session] Fix typo

* [expo-auth-session] Add tests
@brentvatne
Copy link
Member

this is available in expo-auth-session@1.0.1

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.

expo-auth-session does not support release channels due to incorrect defaultReturnUrl
4 participants