Skip to content

Documenting Limit .qgs file edit to admins#696

Merged
SeqLaz merged 2 commits intomasterfrom
QF-7980-limit_qgs_file_edit_to_admins
Mar 3, 2026
Merged

Documenting Limit .qgs file edit to admins#696
SeqLaz merged 2 commits intomasterfrom
QF-7980-limit_qgs_file_edit_to_admins

Conversation

@SeqLaz
Copy link
Member

@SeqLaz SeqLaz commented Feb 27, 2026

For a quick overview of the why:

  • 1 & 2 (Unregistered / Simple Users): ❌ Not allowed.
  • 3 (Project Owner): ✅ Allowed (owners inherently have administrative privileges).
  • 4 (Project Collaborator - Admin): ✅ Allowed.
  • 5, 6, 7, 8 (Manager, Editor, Reporter, Reader): ❌ Not allowed.
  • 9 & 10 (Organization Owner & Admin): ✅ Allowed (these roles sit above the project admin in the hierarchy and retain administrative control over the organization's projects).
  • 11 (Organization Member): ❌ Not allowed.

@SeqLaz SeqLaz requested review from beanzmo and suricactus February 27, 2026 09:25
@duke-nyuki
Copy link

Copy link
Collaborator

@suricactus suricactus left a comment

Choose a reason for hiding this comment

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

But this is only valid when the checkbox is checked, not by default, no?

I would suggest we move this line right after "Upload project's files (qfieldsync)" and rephrase it to:

Upload/delete resticted project's files (.qgs/.qgz/plugins) when "Restrict project files" is checked

@SeqLaz
Copy link
Member Author

SeqLaz commented Feb 27, 2026

Upload/delete resticted project's files (.qgs/.qgz/plugins) when "Restrict project files" is checked

Done!.... Many thanks for the suggestion:

This is how looks:

image

@SeqLaz SeqLaz requested a review from suricactus February 27, 2026 16:08
@SeqLaz SeqLaz merged commit 98b34b6 into master Mar 3, 2026
3 checks passed
@SeqLaz SeqLaz deleted the QF-7980-limit_qgs_file_edit_to_admins branch March 3, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants