Skip to content

Conversation

@Shivam-nagar23
Copy link
Member

@Shivam-nagar23 Shivam-nagar23 commented Sep 30, 2024

Description

Fixes #

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 1, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
5.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@Shivam-nagar23 Shivam-nagar23 merged commit b0731dc into release-candidate-v0.19.0 Oct 1, 2024
@Shivam-nagar23 Shivam-nagar23 deleted the fix-user-apis branch October 1, 2024 15:53
Shivam-nagar23 added a commit that referenced this pull request Oct 1, 2024
* Revert "fix: user flows permissions fix (#5917)"

This reverts commit c869fce.

* user permission updated

* group grouo

* refactoring

* get update

* auth pass duplicate

* chart group

* super admin downgrade

* isactionSuperAdmin check

* fixed job trigger bug

* fixed job trigger bug

* remove break and return on unauthorised

* role group delete fix

* rg fix

---------

Co-authored-by: kartik-579 <[email protected]>
Shivam-nagar23 added a commit that referenced this pull request Oct 1, 2024
* Revert "fix: user flows permissions fix (#5917)"

This reverts commit c869fce.

* user permission updated

* group grouo

* refactoring

* get update

* auth pass duplicate

* chart group

* super admin downgrade

* isactionSuperAdmin check

* fixed job trigger bug

* fixed job trigger bug

* remove break and return on unauthorised

* role group delete fix

* rg fix

---------

Co-authored-by: kartik-579 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants