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

Users who do not have a PL >= to state_default are unable to live share their location #7612

Open
kieranlane opened this issue Jun 29, 2023 · 2 comments
Open

Users who do not have a PL >= to state_default are unable to live share their location #7612

kieranlane opened this issue Jun 29, 2023 · 2 comments
Labels
A-Location-Sharing O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Labs

Comments

@kieranlane
Copy link

This is the iOS equivalent issue to: element-hq/element-android#8563

When attempting to start Live Location Sharing, the client does not use the set permission level "org.matrix.msc3672.beacon_info": 0 or "m.beacon_info": 0. Instead only changes to the state_default allow permission to do so. This means users are unable to share locations unless they have enough PL to send any state.
@Velin92 Velin92 added S-Minor Impairs non-critical functionality or suitable workarounds exist S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Jun 29, 2023
@kittykat kittykat added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Jun 30, 2023
@NicolasBuquet
Copy link

Yes, the power level is not correctly checked when trying to share live location.

Will be fixed when I merge this Tchap PR in Element iOS : tchapgouv/tchap-ios#1006

@NicolasBuquet
Copy link

PR pushed : #7808

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Location-Sharing O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Labs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kittykat @kieranlane @NicolasBuquet @Velin92