Skip to content

Use try/catch to access unauthenticated#6358

Merged
Tobbe merged 2 commits intoredwoodjs:mainfrom
mcmx:fix-route--set-unauth-error-msg
Sep 7, 2022
Merged

Use try/catch to access unauthenticated#6358
Tobbe merged 2 commits intoredwoodjs:mainfrom
mcmx:fix-route--set-unauth-error-msg

Conversation

@mcmx
Copy link
Copy Markdown
Contributor

@mcmx mcmx commented Sep 6, 2022

Fix the bug mentioned in #6357 about a non-descriptive message when using a non-defined route for the unauthenticated prop of Private

@Tobbe Tobbe added the release:fix This PR is a fix label Sep 7, 2022
@Tobbe Tobbe assigned Tobbe and unassigned dthyresson Sep 7, 2022
@Tobbe Tobbe merged commit 5a7fbbb into redwoodjs:main Sep 7, 2022
@redwoodjs-bot redwoodjs-bot Bot added this to the next-release milestone Sep 7, 2022
@Tobbe
Copy link
Copy Markdown
Contributor

Tobbe commented Sep 7, 2022

Thanks for this fix @mcmx! Great work 🙂

jtoar pushed a commit that referenced this pull request Sep 8, 2022
jtoar pushed a commit that referenced this pull request Sep 8, 2022
dac09 added a commit to dac09/redwood that referenced this pull request Sep 9, 2022
* 'main' of github.com:redwoodjs/redwood:
  fix(deps): update dependency @graphql-yoga/common to v2.12.12 (redwoodjs#6349)
  fix(test-project): revert @redwoodjs/core to rc
  Update yarn.lock
  v2.2.4
  bugfix replace slash in tailwind config on windows (redwoodjs#6203)
  bugfix replace slash in tailwind config on windows (redwoodjs#6203)
  chore(deps): update dependency @testing-library/dom to v8.17.1 (redwoodjs#6351)
  Update yarn.lock
  Use try/catch to access unauthenticated (redwoodjs#6358)
  issue#5852 added windows fix for nodeFileTrace (redwoodjs#6325)
  Handle special props `ref` and `key` in path and search params (redwoodjs#5537)
  Use try/catch to access unauthenticated (redwoodjs#6358)
  feat(codemod): Add codemod to make relation resolvers partial (redwoodjs#6342)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:fix This PR is a fix

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants