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

Remove React-native entry from @magic-ext/auth Package.json #595

Merged
merged 3 commits into from
Jul 26, 2023

Conversation

Ariflo
Copy link
Contributor

@Ariflo Ariflo commented Jul 26, 2023

πŸ“¦ Pull Request

Similar to what was done on commit 4b3a106, we need to remove this attribute to permit the extension to work with our RN packages.

βœ… Fixed Issues

n/a

🚨 Test instructions

[Describe any additional context required to test the PR/feature/bug fix.]

⚠️ Don't forget to add a semver label!

Please only add one label:

  • patch: Bug Fix?
  • minor: New Feature?
  • major: Breaking Change?
  • skip-release: It's unnecessary to publish this change.
πŸ“¦ Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @magic-ext/[email protected]
# or 
yarn add @magic-ext/[email protected]

@Ariflo Ariflo added the patch Increment the patch version when merged label Jul 26, 2023
@Ariflo Ariflo requested review from Dizigen and Ethella July 26, 2023 21:17
@Ariflo Ariflo self-assigned this Jul 26, 2023
@shortcut-integration
Copy link

@Ethella
Copy link
Member

Ethella commented Jul 26, 2023

Could we remove that entry point entirely in the codebase, including the scaffold

@Ariflo
Copy link
Contributor Author

Ariflo commented Jul 26, 2023

Could we remove that entry point entirely in the codebase, including the scaffold

@Ethella I thought we did when you did this 4b3a106 πŸ€”

@Ariflo
Copy link
Contributor Author

Ariflo commented Jul 26, 2023

@Ethella, just realized you didn't in that commit. I'll add it to this PR. One sec πŸƒ πŸ’¨

@Ethella
Copy link
Member

Ethella commented Jul 26, 2023

Oh, it's a different Jerry, I'm pretty sure.

Ethella
Ethella previously approved these changes Jul 26, 2023
@Ariflo Ariflo added this pull request to the merge queue Jul 26, 2023
Merged via the queue into master with commit 54b9609 Jul 26, 2023
4 checks passed
@Ariflo Ariflo deleted the ariflo-sc-82632-remove-react-native-from-auth-package branch July 26, 2023 22:04
@Ariflo Ariflo added the released This issue/pull request has been released. label Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants