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.

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

fix(permission): global project roles cannot view/execute/approve tickets #4117

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

MarkPotato777
Copy link
Collaborator

What type of PR is this?

type-bug

What this PR does / why we need it:

This PR fixed several bugs related to the global project roles. The main update of this PR is that I changed all UserResourceRoleRepository references to ResourceRoleService references, which adds global role business logic.
The main reason is that I did not adapted the permission part entirely in the previous code.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:


@MarkPotato777 MarkPotato777 added the type-bug Something isn't working label Jan 2, 2025
@MarkPotato777 MarkPotato777 added this to the ODC 4.3.3 milestone Jan 2, 2025
@MarkPotato777 MarkPotato777 self-assigned this Jan 2, 2025
Copy link
Collaborator

@PeachThinking PeachThinking left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Collaborator

@zijiacj zijiacj left a comment

Choose a reason for hiding this comment

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

lgtm

@MarkPotato777 MarkPotato777 merged commit 7051c34 into dev/4.3.x Jan 3, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug Something isn't working
Projects
Status: Developed
Development

Successfully merging this pull request may close these issues.

4 participants