Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

String PLs are treated as if they were ints #16018

Closed
Sorunome opened this issue Dec 22, 2020 · 0 comments · Fixed by matrix-org/matrix-react-sdk#6245
Closed

String PLs are treated as if they were ints #16018

Sorunome opened this issue Dec 22, 2020 · 0 comments · Fixed by matrix-org/matrix-react-sdk#6245
Labels
A-User-Info P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Spec-Compliance An area where Element doesn't correctly implement the spec

Comments

@Sorunome
Copy link

Description

If a user has a string as PL in the state event, e.g. "100", instead of 100 the UI incorrectly displays they were admin etc.

Steps to reproduce

  • Give a user a string PL

Version information

  • Platform: electron

For the desktop app:

  • OS: Arch
  • Version: 1.7.15
@jryans jryans added defect P2 S-Minor Impairs non-critical functionality or suitable workarounds exist A-User-Info labels Dec 22, 2020
@t3chguy t3chguy added the Z-Spec-Compliance An area where Element doesn't correctly implement the spec label Dec 29, 2020
@jryans jryans removed the defect label Mar 4, 2021
@aaronraimist aaronraimist mentioned this issue Jun 23, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-User-Info P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Spec-Compliance An area where Element doesn't correctly implement the spec
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Do not honor string power levels aaronraimist/matrix-react-sdk
3 participants
@jryans @t3chguy @Sorunome