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

chore(auth): move auth providers into separate packages #7019

Merged
merged 26 commits into from
Dec 6, 2022

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Dec 2, 2022

Follow up to #6922 and similar PRs. I think the names and dependencies of all these packages are going to need more thought, but we can do that after.

@jtoar jtoar added the release:chore This PR is a chore (means nothing for users) label Dec 2, 2022
@jtoar jtoar requested a review from Tobbe December 2, 2022 21:21
@jtoar jtoar self-assigned this Dec 2, 2022
@jtoar jtoar force-pushed the ds-move-other-auth-providers branch from a28c94c to 5778544 Compare December 2, 2022 22:19
@jtoar
Copy link
Contributor Author

jtoar commented Dec 3, 2022

Just noting that I think the smoke tests won't pass till I actually publish the dbAuth packages. I'll confirm

@jtoar jtoar marked this pull request as draft December 3, 2022 04:50
@jtoar jtoar force-pushed the ds-move-other-auth-providers branch from 5778544 to 8830038 Compare December 5, 2022 22:22
@jtoar jtoar marked this pull request as ready for review December 6, 2022 03:13
@jtoar jtoar merged commit bcaef02 into main Dec 6, 2022
@jtoar jtoar deleted the ds-move-other-auth-providers branch December 6, 2022 06:09
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Dec 6, 2022
@redwoodjs-bot
Copy link

redwoodjs-bot bot commented Dec 6, 2022

🔔 @jtoar, @Tobbe—I couldn't cherry pick this one. If you want it in the next release, you'll have to cherry pick it manually.

@jtoar jtoar modified the milestones: next-release, v4.0.0 Dec 8, 2022
dac09 added a commit that referenced this pull request Dec 9, 2022
…xperimental-vite-optin

* 'main' of github.com:redwoodjs/redwood:
  Include dbAuth setup in test project fixture check (#7036)
  fix: dedupe redwoodjs packages (#7050)
  Setup auth: No return on missing modules (#7049)
  fix(deps): update dependency @fastify/static to v6.6.0 (#7029)
  Fix webAuthn import path in setup template (#7041)
  Fix typo in dbauth setup package name (#7040)
  chore(deps): update dependency nx to v15.3.0 (#7037)
  fix: auth provider names in setup auth (#7038)
  fix(deps): update dependency @fastify/url-data to v5.2.0 (#7030)
  fix(deps): update dependency eslint to v8.29.0 (#7031)
  fix(deps): update prisma monorepo to v4.7.1 (#7027)
  fix(deps): update storybook monorepo to v6.5.14 (#7028)
  chore(deps): update dependency @replayio/playwright to v0.3.10 (#7026)
  chore(auth): move auth providers into separate packages (#7019)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants