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

[TASK] Drop duplicate permissions check when (un)hiding events #3150

Merged
merged 1 commit into from
Jul 23, 2024
Merged

[TASK] Drop duplicate permissions check when (un)hiding events #3150

merged 1 commit into from
Jul 23, 2024

Conversation

oliverklee
Copy link
Owner

Now that we're using the DataHandler, we can rely on it doing the permission check for the current BE user.

This reverts commit 45a726d.

@oliverklee
[TASK] Drop duplicate permissions check when (un)hiding events
461bd0e 
Now that we're using the `DataHandler`, we can rely on it
doing the permission check for the current BE user.

This reverts commit 45a726d.
@oliverklee oliverklee self-assigned this Jul 23, 2024
@oliverklee oliverklee enabled auto-merge (squash) July 23, 2024 16:40
@oliverklee oliverklee merged commit 8bffdee into main Jul 23, 2024
62 checks passed
@oliverklee oliverklee deleted the task/drop-access-check branch July 23, 2024 16:45
@coveralls
Copy link

Coverage Status

coverage: 87.752% (-0.005%) from 87.757%
when pulling 461bd0e on task/drop-access-check
into 1f6fe1b on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@oliverklee oliverklee

Labels
cleanup
Projects
DGE
Status: Done
Milestone
5.6.0: New features, lots of internal...
Development

Successfully merging this pull request may close these issues.
2 participants
@oliverklee @coveralls