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.

Sign up for GitHub

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

Jump to bottom

useRequireAuth: Refactor to make it easier to read #4738

Merged
merged 1 commit into from
Mar 13, 2022
Merged

useRequireAuth: Refactor to make it easier to read #4738

merged 1 commit into from
Mar 13, 2022

Conversation

Tobbe
Copy link
Member

@Tobbe Tobbe commented Mar 13, 2022

I was reading this code to try to figure out how useRequireAuth works. While reading it I made some changes that, imho, made it easier to read the code.

@Tobbe Tobbe added the release:chore This PR is a chore (means nothing for users) label Mar 13, 2022
@Tobbe Tobbe requested a review from dthyresson March 13, 2022 00:47
@Tobbe Tobbe self-assigned this Mar 13, 2022
@Tobbe Tobbe merged commit 5e43ab7 into redwoodjs:main Mar 13, 2022
@Tobbe Tobbe deleted the tobbe-userequireauth-refactor branch March 13, 2022 13:50
@jtoar jtoar added this to the next-release milestone Mar 13, 2022
@thedavidprice thedavidprice modified the milestones: next-release, v0.50.0 Mar 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dthyresson dthyresson dthyresson approved these changes

Assignees

@Tobbe Tobbe

Labels
release:chore This PR is a chore (means nothing for users)
Projects
No open projects
Release
Status: Archived
Milestone
v0.50.0
Development

Successfully merging this pull request may close these issues.
4 participants
@Tobbe @dthyresson @thedavidprice @jtoar