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

Permissions evaluation doesn't work for values >=1000 #14006

Closed
osmaczko opened this issue Mar 17, 2024 · 1 comment · Fixed by #14249
Closed

Permissions evaluation doesn't work for values >=1000 #14006

osmaczko opened this issue Mar 17, 2024 · 1 comment · Fixed by #14249
Assignees
@igor-sirotin
Labels
bug Something isn't working E:Desktop Comm Perms and Minting MVP Misc tasks about Community permissions that are not part of another Epic, due for the MVP
Milestone
2.29.0 Beta

Comments

@osmaczko
Copy link
Contributor

Bug Report

Description

Once the permissions value exceeds 999, the permissions evaluation doesn't work.

Steps to reproduce

See video:

parmissions_decimals_issue-2024-03-17_15.22.34.mp4

Expected behavior

Member does't satisfy the criteria and is kicked from channel.

Actual behavior

Member does satisfy the criteria and stays on channel.

Additional Information

  • Status desktop version: master (79fa379)
  • Operating System: Fedora
@osmaczko osmaczko added the bug Something isn't working label Mar 17, 2024
@caybro caybro added the E:Desktop Comm Perms and Minting MVP Misc tasks about Community permissions that are not part of another Epic, due for the MVP label Mar 18, 2024
@caybro
Copy link
Member

caybro commented Mar 18, 2024

Might be related to #11918 ?

@jrainville jrainville added this to the 2.29.0 Beta milestone Mar 18, 2024
@jrainville jrainville mentioned this issue Mar 20, 2024
Closed
@jrainville jrainville moved this to Iteration Backlog in Status Desktop/Mobile Board Mar 26, 2024
@igor-sirotin igor-sirotin moved this from Iteration Backlog to In Progress in Status Desktop/Mobile Board Mar 27, 2024
@igor-sirotin igor-sirotin mentioned this issue Apr 2, 2024
Merged
@igor-sirotin igor-sirotin moved this from In Progress to Code review in Status Desktop/Mobile Board Apr 2, 2024
@github-project-automation github-project-automation bot moved this from Code review to Done in Status Desktop/Mobile Board Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@igor-sirotin igor-sirotin

Labels
bug Something isn't working E:Desktop Comm Perms and Minting MVP Misc tasks about Community permissions that are not part of another Epic, due for the MVP
Projects
Status Desktop/Mobile Board
Archived in project
Milestone
2.29.0 Beta
Development

Successfully merging a pull request may close this issue.

fix: permissions with value>1000 status-im/status-desktop
4 participants
@caybro @jrainville @igor-sirotin @osmaczko