[PM-15970] Allow assigning collections if user has correct permissions

[SaintPatrck]

🎟️ Tracking

PM-15970
PM-15969

📔 Objective

Allows assigning items to collections if the user has manage permissions and is not restricted from viewing or editing password in the collection. This fixes an issue where collection assignment would be blocked if the user could view the item and its passwords.

📸 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
• Used internationalization (i18n) for all UI strings
• CI builds passed
• Communicated to DevOps any deployment requirements
• Updated any necessary documentation 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

[github-actions]

Checkmarx One – Scan Summary & Details – b65ad2a8-1091-4a22-aff9-3486e51f4c2c

No New Or Fixed Issues Found

[SaintPatrck]

@dseverns-livefront @david-livefront This seemed like the best package for the enum, but I'm open to suggestion if there's a more appropriate place.

[dseverns-livefront]

makes sense to me, another thought I had was: if we are not using this outside the extensions it could live in that file and maybe annotate with @VisibleForTesting so if we were to try and use it outside there or the test we get a lint warning, but that may be more philosophical.

TLDR: Placement makes sense to me.

[dseverns-livefront]

👍

[dseverns-livefront]

Just a small typo and a visibility option. But otherwise LGTM!

[codecov]

Codecov Report

Attention: Patch coverage is 82.35294% with 3 lines in your changes missing coverage. Please review.

Project coverage is 88.91%. Comparing base (3329dfa) to head (5f9ca1a).
Report is 6 commits behind head on main.

Files with missing lines	Patch %	Lines
.../ui/vault/feature/util/CollectionViewExtensions.kt	82.35%	0 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4461      +/-   ##
==========================================
- Coverage   88.91%   88.91%   -0.01%     
==========================================
  Files         455      455              
  Lines       39558    39568      +10     
  Branches     5605     5614       +9     
==========================================
+ Hits        35173    35180       +7     
  Misses       2428     2428              
- Partials     1957     1960       +3     

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

[dseverns-livefront]