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

Modify access control for fetching all notification templates #1387

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

KizerovDmitriy
Copy link
Contributor

@KizerovDmitriy KizerovDmitriy commented Jun 27, 2024

GreenCityUBS PR

This pull request modifies the access control for the endpoint that fetches all notification templates.

Summary Of Changes 🔥

  • Updated @PreAuthorize annotation to use @preAuthorizer.hasAuthority('SEE_MESSAGES_PAGE', authentication) for the GET /get-all-templates endpoint.

Issue Link

#6349

PR Checklist Forms

(to be filled out by PR submitter)

  • Code is up-to-date with the prod branch.
  • You've successfully built and run the tests locally.
  • There are new or updated unit tests validating the change.
  • JIRA/ Github Issue number & title in PR title (ISSUE-XXXX: Ticket title)
  • This template filled (above this section).
  • Sonar's report does not contain bugs, vulnerabilities, security issues, code smells or duplication
  • NEED_REVIEW and READY_FOR_REVIEW labels are added.
  • All files reviewed before sending to reviewers

@KizerovDmitriy KizerovDmitriy self-assigned this Jun 27, 2024
@KizerovDmitriy KizerovDmitriy added the bug Something isn't working label Jun 27, 2024
@KizerovDmitriy KizerovDmitriy merged commit c0e984d into prod Jun 27, 2024
@KizerovDmitriy KizerovDmitriy deleted the modify-access-to-notification-templates branch June 27, 2024 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants