Skip to content
This repository was archived by the owner on Apr 12, 2022. It is now read-only.

Set user id before storing it#558

Merged
yostyle merged 3 commits into
developfrom
yostyle/bugfix_user_preference_crash
Sep 22, 2020
Merged

Set user id before storing it#558
yostyle merged 3 commits into
developfrom
yostyle/bugfix_user_preference_crash

Conversation

@yostyle
Copy link
Copy Markdown
Contributor

@yostyle yostyle commented Sep 22, 2020

Fix a crash when it checks user preference (related to matrix-org/synapse#7606)

@yostyle yostyle changed the title set user id before storing it Set user id before storing it Sep 22, 2020
Copy link
Copy Markdown
Contributor

@bmarty bmarty left a comment

Choose a reason for hiding this comment

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

one remark, else LGTM

Comment thread CHANGES.rst Outdated

Bugfix:
-
- Fix a crash when it checks user preference (related to matrix-org/synapse#7606).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

preference, did you mean presence?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

you right, I will update this.

@yostyle yostyle merged commit 3d3c42c into develop Sep 22, 2020
@bmarty bmarty deleted the yostyle/bugfix_user_preference_crash branch September 22, 2020 12:42
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