Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fix issues with the wallet locking #1095

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

ost-ptk
Copy link
Member

@ost-ptk ost-ptk commented Nov 22, 2024

Make sure to fill in all the below sections.

Description

  • Introduce initKeepAlive to manage background script activity and prevent session termination when the vault is unlocked. Implement the keepAlive function to periodically ping the script, with methods to start and stop based on the vault state.
  • Reverted logic for timeout vault lock mechanism.
  • Added condition for Bring to work only for Chrome.

Linked tickets

WALLET-446
WALLET-447
WALLET-452

Checklist

  • Make sure this PR title follows semantic release conventions: https://semantic-release.gitbook.io/semantic-release/#commit-message-format

  • If the PR adds any new text to the UI, make sure they are localized

  • Include a screenshot or recording if implementing significant UI or user flow change

  • When this PR affects architecture changes wait for review from Dmytro before merging

@ost-ptk ost-ptk requested a review from Comp0te November 22, 2024 14:16
@Comp0te Comp0te merged commit 56d4be4 into develop Dec 9, 2024
5 checks passed
@Comp0te Comp0te deleted the WALLET-446-cw-chrome-wallet-is-locked-prematurely branch December 9, 2024 21:30
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