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

fix(og-mw-setup): Fix typo in notes for og-middleware setup #10736

Merged
merged 2 commits into from
Jun 5, 2024

Conversation

dac09
Copy link
Collaborator

@dac09 dac09 commented Jun 5, 2024

Found a typo when I tried this earlier, I think it the setup instructions should ask users to install on the web side, not the api :)

@dac09 dac09 added the release:fix This PR is a fix label Jun 5, 2024
@dac09 dac09 requested a review from cannikin June 5, 2024 16:01
@dac09 dac09 self-assigned this Jun 5, 2024
@dac09 dac09 added the changesets-ok Override the changesets check label Jun 5, 2024
@dac09 dac09 added this to the SSR milestone Jun 5, 2024
@cannikin
Copy link
Member

cannikin commented Jun 5, 2024

Good catch! I think there are some code things to update as well, when I used this command with the latest RSC updates I remember something not getting configured correctly and then having to go back in and modify by hand...

…ler.ts

Co-authored-by: Rob Cameron <cannikin@fastmail.com>
@dac09
Copy link
Collaborator Author

dac09 commented Jun 5, 2024

not getting configured correctly and then having to go back in and modify by hand...

I tried it today, seemed to be ok, but if you spot it again let me know, can update this PR or do another one :)

@cannikin cannikin merged commit e6836b4 into redwoodjs:main Jun 5, 2024
50 of 51 checks passed
@Josh-Walker-GM Josh-Walker-GM modified the milestones: SSR, v8.0.0 Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changesets-ok Override the changesets check release:fix This PR is a fix
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants