Skip to content

[PM-40396] tools export routing change from tools to settings - #22416

Open
itsadrago wants to merge 3 commits into
mainfrom
tools/pm-40396/route-change-tools-export
Open

[PM-40396] tools export routing change from tools to settings#22416
itsadrago wants to merge 3 commits into
mainfrom
tools/pm-40396/route-change-tools-export

Conversation

@itsadrago

Copy link
Copy Markdown
Contributor

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-40396

📔 Objective

Relocate the export route from tools/export to settings/export and add an unconditional redirect from the old path to preserve bookmarks, deep links, and docs. The export component itself is unchanged (i.e. no UI changes); only its route location moves.

@itsadrago
itsadrago requested a review from a team as a code owner August 13, 2026 20:35
@itsadrago
itsadrago requested a review from vleague2 August 13, 2026 20:35
@itsadrago itsadrago added t:feature Change Type - Feature Development needs-qa Marks a PR as requiring QA approval labels Aug 13, 2026
[spotlight]="true"
></bit-nav-item>
<app-coachmark #importCoachmark stepId="importData" />
<bit-nav-item [text]="'exportNoun' | i18n" route="tools/export"></bit-nav-item>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this for VFO1? Should it be feature-flagged?

@itsadrago itsadrago Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, I believe so. Good point. Do we need to create a FF or is there an existing one that should be used?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Vault already made one! VFO1Foundation = "vfo1-foundation"

@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.87%. Comparing base (2c83757) to head (2e91f87).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
apps/web/src/app/oss-routing.module.ts 0.00% 9 Missing ⚠️
apps/web/src/app/layouts/user-layout.component.ts 0.00% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #22416      +/-   ##
==========================================
- Coverage   53.88%   53.87%   -0.01%     
==========================================
  Files        4297     4297              
  Lines      136486   136500      +14     
  Branches    21569    21570       +1     
==========================================
+ Hits        73545    73546       +1     
- Misses      57574    57587      +13     
  Partials     5367     5367              

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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 t:feature Change Type - Feature Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants