Closed matt-livefront closed 3 weeks ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.63%. Comparing base (
0b1afe3
) to head (c7b5520
). Report is 4 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One â Scan Summary & Details â 5701be7b-173e-41af-ade2-8a30eb3dfa70
đī¸ Tracking
PM-11275
đ Objective
Updates the app to use the
keyConnectorEnabled
flag instead ofuseKeyConnector
on the organization. We were seeing thatuseKeyConnector
would be incorrectly set at times andkeyConnectorEnabled
is what's used on non-mobile clients.https://github.com/bitwarden/clients/blob/887d1924a331a292c5425136aac39b14166cf814/libs/common/src/auth/services/key-connector.service.ts#L122
â° Reminders before review
đĻŽ 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