Document the nuances in UserId and SessionId types.

[julioromano]

Type of change

• Feature
• Bugfix
• Technical
• Other : Docs

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/HjZQKj

[bmarty]

LGTM.

To add some context, SessionId is used to identify a session, and the value used is the user MatrixId. In the previous SDK we where concatenating the userId and the deviceId in the SDK to let the SDK be able to log in several times with the same user. We do not need to do that anymore, but we like to keep SessionId distinct from UserId.

The SessionId is used to identify the session, for instance when coming from a notification intent. This is preparatory work for multi session support.

[julioromano]

LGTM.

To add some context, SessionId is used to identify a session, and the value used is the user MatrixId. In the previous SDK we where concatenating the userId and the deviceId in the SDK to let the SDK be able to log in several times with the same user. We do not need to do that anymore, but we like to keep SessionId distinct from UserId.

The SessionId is used to identify the session, for instance when coming from a notification intent. This is preparatory work for multi session support.

Does this mean in the feature the semantic meaning of SessionId will change?

Right now is MXID of the current user.
In the future it might become "some string that identifies this session but might not be an MXID" ?

[bmarty]

i do not think it will change, I am just giving some context here :)