Skip to content

Conversation

@ice-linus
Copy link
Contributor

Description

Implemented multi-account support for push notifications. Added push subscription cleanup on logout, notification display after account switching, improved handling of notifications for messages encrypted for other users, and added a method to retrieve pubkey by identity key name with caching.

Additional Notes

  • Notifications are shown even if the message is encrypted for another account (user can switch to view)
  • Added localized strings for account switch notification
  • Caches and databases are invalidated on account switch

Task ID

ION-4646

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation
  • Chore

@ice-linus ice-linus marked this pull request as ready for review December 5, 2025 14:13
@ice-morpheus ice-morpheus enabled auto-merge (squash) December 5, 2025 14:14
@ice-morpheus ice-morpheus force-pushed the feature/switch-accaunts-via-push branch 8 times, most recently from baf3d3f to 2c09a3b Compare December 8, 2025 11:41
@ice-linus ice-linus force-pushed the feature/switch-accaunts-via-push branch from 2c09a3b to 1144556 Compare December 9, 2025 08:57
@ice-morpheus ice-morpheus force-pushed the feature/switch-accaunts-via-push branch 14 times, most recently from 74efbb8 to 8d35674 Compare December 10, 2025 10:50
@ice-morpheus ice-morpheus force-pushed the feature/switch-accaunts-via-push branch 26 times, most recently from c092dc2 to e362d1c Compare December 19, 2025 08:34
- Implemented logic to delete push subscriptions for users upon logout.
- Added state management for showing notifications after switching accounts.
- Enhanced the handling of push notifications to ensure users receive relevant messages.
- Introduced a method to retrieve user public keys based on identity key names for improved account management.
@ice-morpheus ice-morpheus force-pushed the feature/switch-accaunts-via-push branch from e362d1c to 8af7af5 Compare December 19, 2025 09:03
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.

4 participants