Skip to content

Conversation

@kartik-579
Copy link
Member

@kartik-579 kartik-579 commented Apr 10, 2023

Description

Updated condition (dml handling for updated accessType and entity) for getting filtered role filters.

Fixes #3266

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Local Environment
  • QA Cluster

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

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@kartik-579 kartik-579 changed the title fix: removed condition for updating roleFilters when getting user details fix: updated condition for getting role filters Apr 10, 2023
@kripanshdevtron kripanshdevtron self-requested a review April 10, 2023 18:18
@kartik-579 kartik-579 merged commit fd5f896 into main Apr 10, 2023
@kartik-579 kartik-579 deleted the auth-fix branch April 10, 2023 18:47
gireesh-naidu pushed a commit that referenced this pull request Jun 16, 2023
* chore: Discord notification for a specific label ("pager-duty") (#3249)

* new action added

* changes

* changes

---------

Co-authored-by: Kamal Acharya <[email protected]>

* action added (#3265)

Co-authored-by: Kamal Acharya <[email protected]>
Co-authored-by: Prakarsh <[email protected]>

* fix: updated condition for getting role filters  (#3261)

* removed code which was corrupting data for role filters

* updated condition for user data fetch api role filters dml handling

* wip

---------

Co-authored-by: kamal-devtron <[email protected]>
Co-authored-by: Kamal Acharya <[email protected]>
Co-authored-by: Prakarsh <[email protected]>
gireesh-naidu pushed a commit that referenced this pull request Jun 16, 2023
* chore: Discord notification for a specific label ("pager-duty") (#3249)

* new action added

* changes

* changes

---------

Co-authored-by: Kamal Acharya <[email protected]>

* action added (#3265)

Co-authored-by: Kamal Acharya <[email protected]>
Co-authored-by: Prakarsh <[email protected]>

* fix: updated condition for getting role filters  (#3261)

* removed code which was corrupting data for role filters

* updated condition for user data fetch api role filters dml handling

* wip

* fix: updated condition for handling linked ci creation with parent ci docker config overriden (#3243)

* fix for linked ci creation

* fix for linked ci delete panic

* wip

---------

Co-authored-by: kamal-devtron <[email protected]>
Co-authored-by: Kamal Acharya <[email protected]>
Co-authored-by: Prakarsh <[email protected]>
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.

Bug: Unexpected end of json input while adding new permissions for users

3 participants