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

feat: web authentication #335

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: web authentication #335

wants to merge 4 commits into from

Conversation

mikehardy
Copy link
Collaborator

This is the result of rebasing PR #166 from @Romick2005 - it was such a large change and I performed no testing it did not feel respectful to push back to that branch as Romick2005 may be depending on it and I can't vouch for the quality of this rebase

In all other ways, refer to that PR though - only changes here are that I attempted to resolve conflicts, and I bumped the react-native-webview and uuid dependencies that the PR added to their current versions and ran yarn to update lock files accordingly

The way to move this forward is for someone to either pull this PR locally and test it, or to use the test patches that this PR should generate (check the Summary page of the Details link for the patch generate CI job below...) and make sure it works.

All contributions welcome

@miso-belica
Copy link

Hello, thank you all for the implementation. Is "@invertase/react-native-apple-authentication": "git+https://github.com/invertase/react-native-apple-authentication.git#d8561f66aa5383e4bada1c31ea9a24137dffef43", in package.json enough to test this? Or anything else is needed? I could not find the patches you mentioned.

@mikehardy
Copy link
Collaborator Author

mikehardy commented Nov 18, 2023

Hey @miso-belica 👋 ! Apologies, I was confused on which repo I was working in at the moment. Most of my invertase work is in react-native-firebase which does generate a patch-package set since it is a monorepo and there's no other way for users to test things pre-release.

This repository is a regular repo so has no such problem, you should be able to just use the git commit hash as you mention and I think that will be sufficient

Curious to know how it works!

@hyochan
Copy link

hyochan commented Dec 20, 2023

I am eagerly waiting for this !

@mikehardy
Copy link
Collaborator Author

@hyochan please test it using the commit hash as mentioned above and let us know how it goes. I'm waiting to hear some reports on how my rebase worked, as I commented above.

The way to move this forward is for someone to either pull this PR locally and test it, or to use the test patches that this PR should generate (check the Summary page of the Details link for the patch generate CI job below...) and make sure it works.

All contributions welcome

@miso-belica
Copy link

I am sorry guys. My prios changed and I was busy with other work. Now, I have a winter vacation but I want to return to this next year for sure 🙂

@mikehardy mikehardy mentioned this pull request Feb 8, 2024
@gavrilikhin-d
Copy link

This would break web build due to missing AppleButton.web.js.

Also it requires to add additional logic for web login which is sad ;(

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.

5 participants