Skip to content

Open widget window on login#1277

Merged
dajimenezriv-internxt merged 7 commits intofeat/2-6-6-releasefrom
fix-frontend-process-issues
Feb 19, 2026
Merged

Open widget window on login#1277
dajimenezriv-internxt merged 7 commits intofeat/2-6-6-releasefrom
fix-frontend-process-issues

Conversation

@dajimenezriv-internxt
Copy link
Contributor

@dajimenezriv-internxt dajimenezriv-internxt commented Feb 17, 2026

What

  1. Remove the mnemonic property from the electron-store since we are using the one inside userData.
  2. Fix some issues when showing and hiding the frontend.

@dajimenezriv-internxt dajimenezriv-internxt self-assigned this Feb 17, 2026
* the usage of device.bucket.
*/
export type User = Omit<UserResponseDto, 'privateKey' | 'backupsBucket'> & { needLogout?: boolean; privateKey: string };
export type User = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We are going to limit the exact properties that can be accessible through the codebase. We will do the same for stored information in the config.json, so we don't store things in plain text and we manage more carefully every property that comes from the user object.

})
.catch((exc) => logger.error({ msg: 'Error starting app', exc }));

eventBus.on('USER_LOGGED_IN', () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We have moved this inside the emitUserLoggedIn function.

@sonarqubecloud
Copy link

❌ The last analysis has failed.

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
2.6% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@dajimenezriv-internxt dajimenezriv-internxt merged commit fba6eb4 into feat/2-6-6-release Feb 19, 2026
8 of 9 checks passed
@dajimenezriv-internxt dajimenezriv-internxt deleted the fix-frontend-process-issues branch February 19, 2026 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants