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

[PM 4983] migrate vault timeout input component #8661

Merged
merged 3 commits into from
May 22, 2024

Conversation

vinith-kovan
Copy link
Collaborator

Type of change

- [ ] Bug fix
- [ ] New feature development
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc)
- [ ] Build/deploy pipeline (DevOps)
- [X] Other

Objective

Migrate the Vault timeout input component to use the component library.

Code changes

vault-timeout-input.component.html - Updated the inputs, and select options to use the equivalent components from the Component Library.

Screenshots

preference.mp4

@github-actions github-actions bot added the needs-qa Marks a PR as requiring QA approval label Apr 10, 2024
Copy link

codecov bot commented Apr 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.20%. Comparing base (ace524c) to head (a4baa6b).

Current head a4baa6b differs from pull request most recent head ab85f47

Please upload reports for the commit ab85f47 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8661      +/-   ##
==========================================
- Coverage   28.15%   27.20%   -0.96%     
==========================================
  Files        2369     2339      -30     
  Lines       70028    68267    -1761     
  Branches    13160    12749     -411     
==========================================
- Hits        19718    18570    -1148     
+ Misses      48751    48301     -450     
+ Partials     1559     1396     -163     

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

Copy link
Contributor

github-actions bot commented Apr 10, 2024

Logo
Checkmarx One – Scan Summary & Detailsdca0da34-3a97-4035-9ec1-6392d183a9f3

No New Or Fixed Issues Found

@vinith-kovan vinith-kovan requested a review from a team as a code owner April 11, 2024 11:50
@vinith-kovan vinith-kovan force-pushed the PM-4983-Migrate-vault-timeout-input-component branch from 8dd4a7a to a4baa6b Compare April 11, 2024 12:12
@vincentsalucci vincentsalucci requested review from eliykat and removed request for vincentsalucci April 25, 2024 21:12
@eliykat eliykat requested review from a team and ike-kottlowski and removed request for a team and eliykat April 25, 2024 22:45
@eliykat
Copy link
Member

eliykat commented Apr 25, 2024

I've requested a review from Auth Team as they should be code owners and they're also the team listed on the Jira ticket.

Copy link
Contributor

@ike-kottlowski ike-kottlowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

verified changes locally.

@vinith-kovan vinith-kovan merged commit 588663f into main May 22, 2024
31 of 32 checks passed
@vinith-kovan vinith-kovan deleted the PM-4983-Migrate-vault-timeout-input-component branch May 22, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-qa Marks a PR as requiring QA approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants