Skip to content

[PM-24795] When logged out, "(empty)" options appear under File#19982

Merged
jengstrom-bw merged 7 commits into
mainfrom
vault/pm-24795/when-logged-out-empty-options-appear-under-File
Apr 10, 2026
Merged

[PM-24795] When logged out, "(empty)" options appear under File#19982
jengstrom-bw merged 7 commits into
mainfrom
vault/pm-24795/when-logged-out-empty-options-appear-under-File

Conversation

@jengstrom-bw
Copy link
Copy Markdown
Contributor

🎟️ Tracking

Jira

📔 Objective

Fix Lock Vault and Log Out menu item enabled state when there are no lockable or authenticated accounts

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.72%. Comparing base (fc06f38) to head (38dc57c).
⚠️ Report is 115 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #19982      +/-   ##
==========================================
+ Coverage   46.68%   46.72%   +0.04%     
==========================================
  Files        3880     3879       -1     
  Lines      116128   116085      -43     
  Branches    17674    17678       +4     
==========================================
+ Hits        54215    54246      +31     
+ Misses      59444    59372      -72     
+ Partials     2469     2467       -2     

☔ View full report in Codecov by Sentry.
📢 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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 3, 2026

Logo
Checkmarx One – Scan Summary & Details5a51f640-ba0b-499a-821d-dda62f5833d7

Great job! No new security vulnerabilities introduced in this pull request

@jengstrom-bw jengstrom-bw marked this pull request as ready for review April 7, 2026 15:34
Copy link
Copy Markdown
Member

@shane-melton shane-melton left a comment

Choose a reason for hiding this comment

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

Logic changes look good, I'd just suggest we add some tests to prevent this from happening again in the future.

Comment thread apps/desktop/src/main/menu/menu.first.ts Outdated
Comment thread apps/desktop/src/main/menu/menu.first.ts
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 9, 2026

@jengstrom-bw
Copy link
Copy Markdown
Contributor Author

Added tests and removed unneeded visible

Copy link
Copy Markdown
Member

@shane-melton shane-melton left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🎉 Thanks!

@jengstrom-bw jengstrom-bw merged commit bd2b4e8 into main Apr 10, 2026
93 checks passed
@jengstrom-bw jengstrom-bw deleted the vault/pm-24795/when-logged-out-empty-options-appear-under-File branch April 10, 2026 16:00
@djsmith85 djsmith85 added t:bugfix Change Type - Bugfix desktop Desktop Application labels Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Desktop Application t:bugfix Change Type - Bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants