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

[Backport 2.x] Adds the newly created admin api permissions to the static dropdown list #1447

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport e6c63ff from #1426

…ist (#1426)

* Adds the newly created admin api permissions to the static dropdown of permissions displayed when creating/modifying a role

---------

Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
Co-authored-by: Ryan Liang <109499885+RyanL1997@users.noreply.github.com>
(cherry picked from commit e6c63ff)
@DarshitChanpura
Copy link
Member

this PR requires opensearch-project/security#2411 to be backported to 2.x

@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Merging #1447 (d921a2b) into 2.x (c83f5d4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.x    #1447   +/-   ##
=======================================
  Coverage   65.51%   65.51%           
=======================================
  Files          93       93           
  Lines        2294     2294           
  Branches      312      312           
=======================================
  Hits         1503     1503           
  Misses        723      723           
  Partials       68       68           
Impacted Files Coverage Δ
public/apps/configuration/constants.tsx 86.20% <ø> (ø)

@cwperks
Copy link
Member

cwperks commented Jun 16, 2023

Closing in favor of: #1473

@cwperks cwperks closed this Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants