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

impr: optimize permissions middleware (@fehmer) #5801

Merged
merged 1 commit into from
Aug 20, 2024
Merged

impr: optimize permissions middleware (@fehmer) #5801

merged 1 commit into from
Aug 20, 2024

Conversation

fehmer
Copy link
Member

@fehmer fehmer commented Aug 18, 2024

Use getPartialUser instead of getUser

@monkeytypegeorge monkeytypegeorge added the backend Server stuff label Aug 18, 2024
@github-actions GitHub Actions
Copy link
Contributor

Continuous integration check(s) failed. Please review the failing check's logs and make the necessary changes.

@fehmer fehmer force-pushed the feature/optimize-permissions-middleware branch from bd747e3 to 8886a51 Compare August 18, 2024 22:05
Miodec
Miodec approved these changes Aug 20, 2024
View reviewed changes
Copy link
Member

@Miodec Miodec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@Miodec Miodec merged commit 4466acd into master Aug 20, 2024
10 checks passed
@Miodec Miodec deleted the feature/optimize-permissions-middleware branch August 20, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Miodec Miodec Miodec approved these changes

Assignees
No one assigned
Labels
backend Server stuff
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fehmer @Miodec @monkeytypegeorge