Skip to content

feat: add @auth/astro framework library with example & docs#6425

Closed
TheOtterlord wants to merge 0 commit intonextauthjs:mainfrom
TheOtterlord:main
Closed

feat: add @auth/astro framework library with example & docs#6425
TheOtterlord wants to merge 0 commit intonextauthjs:mainfrom
TheOtterlord:main

Conversation

@TheOtterlord
Copy link

NOTE:

  • It's a good idea to open an issue first to discuss potential changes.
  • Please make sure that you are NOT opening a PR to fix a potential security vulnerability. Instead, please follow the Security guidelines to disclose the issue to us confidentially.

☕️ Reasoning

🚀 Introduces an Astro framework package, with documentation & an example. The package exposes Astro components in addition to the methods.

Thanks to @tajkirkpatrick and @nowaythatworked who've helped out with building, debugging & documenting this package.

🧢 Checklist

  • Documentation
  • Tests
  • Ready to be merged

🎫 Affected issues

Please scout and link issues that might be solved by this PR.

Fixes: INSERT_ISSUE_LINK_HERE

📌 Resources

@vercel
Copy link

vercel bot commented Jan 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
auth-docs ❌ Failed (Inspect) Jan 21, 2023 at 2:57PM (UTC)
1 Ignored Deployment
Name Status Preview Comments Updated
next-auth-docs ⬜️ Ignored (Inspect) Jan 21, 2023 at 2:57PM (UTC)

@vercel
Copy link

vercel bot commented Jan 17, 2023

@TheOtterlord is attempting to deploy a commit to the authjs Team on Vercel.

A member of the Team first needs to authorize it.

@TheOtterlord
Copy link
Author

We've made a few fixes & changes since I opened the pull request, and with the latest commit this new package now supports node 17.4.0 and higher. Overall the package is a lot more stable now, and is ready for review.

Can't wait to hear the feedback from the team & get this merged! 🚀

@TheOtterlord
Copy link
Author

I've deployed the included example application to https://auth-astro-example.vercel.app showing what it looks like.

@TheOtterlord
Copy link
Author

Closed automatically when fixing merge conflicts -_-

See #6463 for the up-to-date PR

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.

1 participant