Skip to content

Use the modern multi-account session handling (and transition old sessions in)#2369

Closed
GigiaJ wants to merge 7 commits intocinnyapp:devfrom
GigiaJ:modern-sess
Closed

Use the modern multi-account session handling (and transition old sessions in)#2369
GigiaJ wants to merge 7 commits intocinnyapp:devfrom
GigiaJ:modern-sess

Conversation

@GigiaJ
Copy link
Contributor

@GigiaJ GigiaJ commented Jun 21, 2025

Description

Important, but this should be merged after #2366 and the requirements it has.
It isn't a hard requirement though and the merge conflicts relatively will be mild.

Adds the multi-account session handling in its entirety and removes legacy references.
Shouldn't impact users at all, but allows me to move forward with better interfacing with the local storage. (Suffering a good number of issues with asynchronously doing so as well as session states fail after refresh on modifying it. This solves that.)

No actual interface for multiple accounts, but in theory they should be doable now.

Fixes #

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [o] This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@kfiven
Copy link
Collaborator

kfiven commented Mar 14, 2026

We do not have this in roadmap yet. Will revisit once we move over to new SDK.

@kfiven kfiven closed this Mar 14, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Mar 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants