Skip to content

Conversation

@arimendelow
Copy link
Contributor

This PR:

  • Adds a link to the dbAuth OAuth plugin
  • Fixes some links in the Storybook docs

@arimendelow arimendelow changed the title chore(docs) chore(docs): Fix links in Storybook docs; add link to dbAuth OAuth plugin Sep 17, 2024
@Philzen
Copy link
Contributor

Philzen commented Jan 7, 2025

@Tobbe This looks pretty good to me, can you merge?

These changes were probably intended for the v8 docs (definitely are relevant to it), but i'm not sure i can add to this PR, so i may do it in a separate one.

@arimendelow
Copy link
Contributor Author

@Tobbe This looks pretty good to me, can you merge?

These changes were probably intended for the v8 docs (definitely are relevant to it), but i'm not sure i can add to this PR, so i may do it in a separate one.

Yup intended for the v8 docs :) related to your RFC for sure, wanted to keep the links from breaking.

If adding to this PR is somehow easier than creating a new one, let me know and we can figure out how to make that happen.

Thanks for taking on this link fixing!

@Philzen
Copy link
Contributor

Philzen commented Jan 10, 2025

If adding to this PR is somehow easier

i'm afraid i don't have those permissions (or is there another way?)

However – i had already forgotten today about this PR when i did this one this afternoon: #11892

I'll just rebase on main when this is merged, then everything should be complete for the time being. 😃

@Tobbe Tobbe added the release:docs This PR only updates docs label Jan 15, 2025
@Tobbe Tobbe added this to the next-release-patch milestone Jan 15, 2025
@Tobbe Tobbe enabled auto-merge (squash) January 15, 2025 07:42
@Tobbe Tobbe merged commit ba4f622 into redwoodjs:main Jan 15, 2025
48 of 53 checks passed
@Tobbe Tobbe modified the milestones: next-release-patch, v8.4.3 Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:docs This PR only updates docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants