Skip to content

Multiple UI improvements and bug fixes in Governance feature#922

Merged
Avishka-Shamendra merged 17 commits intomainfrom
feature-governance
Feb 21, 2025
Merged

Multiple UI improvements and bug fixes in Governance feature#922
Avishka-Shamendra merged 17 commits intomainfrom
feature-governance

Conversation

@Piumal1999
Copy link
Copy Markdown
Contributor

@Piumal1999 Piumal1999 commented Feb 19, 2025

Purpose

Multiple UI improvements and bug fixes in governance feature

Approach

  • Handle pending and unapplied states in progress bars
  • Disable sorting in path, message and description columns in RuleViolationSummary
  • Conditionally render compliance page based on API type (Compliance page will be not displayed in API products, GraphQL, SOAP APIs)
  • Add download functionality for ruleset content [1]
  • Limit description field height in AddEditPolicy
  • Display compliance results with pending policies. Now the compliance view will display the completed results even when there are pending policy evaluations. [2]
  • Added info alert for API Create and Update actions in ActionConfigDialog [3]. Fixes Policy Action Restriction in API Create & Update States api-manager#3598 as well with 76007b9.
  • Updated the blocking notification in overview page publish action to have a link to the lifecycle page. [4]
  • Disables edit option in API create/update action configs
  • Improve the rule violation table of blocking scenario [5] (Fixes [Publisher Portal] [Deployments] Unhandled Error When Deploying API Revision with Violated Policy api-manager#3612)
  • Add JSON download functionality for governance violations [5]

Screenshots

[1]
image

[2]
image

[3]
image
image

[4]
image

[5]
image

@Piumal1999 Piumal1999 marked this pull request as ready for review February 19, 2025 07:55
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

@Avishka-Shamendra Avishka-Shamendra merged commit 617a66c into main Feb 21, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants