Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

presence_stream has many nullable fields #16467

Open
clokep opened this issue Oct 11, 2023 · 1 comment
Open

presence_stream has many nullable fields #16467

clokep opened this issue Oct 11, 2023 · 1 comment
Labels
A-Database DB stuff like queries, migrations, new/remove columns, indexes, unexpected entries in the db O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@clokep
Copy link
Member

clokep commented Oct 11, 2023

Every column of presence_stream is nullable, but this doesn't really make sense. All of the columns are required except status_msg, which can be null.
@clokep clokep added S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. O-Uncommon Most users are unlikely to come across this or unexpected workflow labels Oct 11, 2023
@clokep clokep mentioned this issue Oct 11, 2023
Merged
@clokep
Copy link
Member Author

clokep commented Oct 11, 2023

Noticed in #16444 (comment).

@reivilibre reivilibre added the A-Database DB stuff like queries, migrations, new/remove columns, indexes, unexpected entries in the db label Oct 12, 2023
@matrixbot matrixbot mentioned this issue Dec 22, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Database DB stuff like queries, migrations, new/remove columns, indexes, unexpected entries in the db O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Blocks non-critical functionality, workarounds exist. T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@clokep @reivilibre