Skip to content

Fix such that Class Roles are added to Method Roles #148

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
Merged
merged 3 commits into from
Jan 31, 2023
Merged

Fix such that Class Roles are added to Method Roles #148

merged 3 commits into from
Jan 31, 2023

Conversation

SentryMan
Copy link
Collaborator

@SentryMan SentryMan commented Jan 25, 2023 •
edited by rbygrave
Loading

Currently getting OperationUnsupportedException

Edit by Rob

This change is such that the roles for a given method are the roles defined on the method PLUS the roles defined on the controller class / type.

@SentryMan
Copy link
Collaborator Author

09e3c79
9544536
these two commits have broke the client tests

@rbygrave rbygrave changed the title Fix Class Roles Fix such that Class Roles are added to Method Roles Jan 30, 2023
@rbygrave rbygrave added this to the 1.24 milestone Jan 30, 2023
@rbygrave rbygrave merged commit e5c51e9 into avaje:master Jan 31, 2023
@SentryMan SentryMan deleted the branch branch January 31, 2023 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@SentryMan SentryMan

Labels
None yet
Projects
None yet
Milestone
1.24
Development

Successfully merging this pull request may close these issues.
2 participants
@SentryMan @rbygrave