Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[PM-19108] Enhance Privileged Apps List View #4898

Draft
wants to merge 1 commit into
base: innovation-sprint/privileged-apps/settings-mgmt
Choose a base branch
from

Conversation

SaintPatrck
Copy link
Contributor

🎟️ Tracking

PM-19108

📔 Objective

Refactor the Privileged Apps List view to provide a more comprehensive and user-friendly display of trusted applications.

📸 Screenshots

Coming soon!

⏰ Reminders before review

  • Contributor guidelines followed
  • All formatters and local linters executed and passed
  • Written new unit and / or integration tests where applicable
  • Protected functional changes with optionality (feature flags)
  • Used internationalization (i18n) for all UI strings
  • CI builds passed
  • Communicated to DevOps any deployment requirements
  • Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

  • 👍 (:+1:) or similar for great changes
  • 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
  • ❓ (:question:) for questions
  • 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
  • 🎨 (:art:) for suggestions / improvements
  • ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
  • 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
  • ⛏ (:pick:) for minor or nitpick changes

Verified

This commit was signed with the committer’s verified signature. The key has expired.
SaintPatrck Patrick Honkonen
Refactor the Privileged Apps List view to provide a more comprehensive and user-friendly display of trusted applications.
Copy link

codecov bot commented Mar 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.93%. Comparing base (3c1ef13) to head (10d1158).

Additional details and impacted files
@@                                 Coverage Diff                                 @@
##           innovation-sprint/privileged-apps/settings-mgmt    #4898      +/-   ##
===================================================================================
- Coverage                                            83.05%   82.93%   -0.13%     
===================================================================================
  Files                                                  628      629       +1     
  Lines                                                48904    48980      +76     
  Branches                                              6758     6775      +17     
===================================================================================
+ Hits                                                 40619    40620       +1     
- Misses                                                6011     6087      +76     
+ Partials                                              2274     2273       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

github-actions bot commented Mar 19, 2025

Logo
Checkmarx One – Scan Summary & Details19df4f72-b846-4879-97c3-210dbe47a543

Great job, no security vulnerabilities found in this Pull Request

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant