Type of change

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

Objective

We aren't completely removing keytar keys right now, and instead are just nulling out values. We should be completely cleaning up those keys if they are set to null, like on logout..

Code changes

To achieve this I checked for a null value on any secure storage set operations and used the remove method instead of save if the passed in value is null.

Screenshots

Before you submit

• I have checked for linting errors (npm run lint) (required)
• I have added unit tests where it makes sense to do so (encouraged but not required)
• This change requires a documentation update (notify the documentation team)
• This change has particular deployment requirements (notify the DevOps team)