Page MenuHomePhabricator

Finalize permissions for submitting emergency reports
Closed, ResolvedPublic

Description

Prior to releasing the emergency report flow for IRS, we need to establish who is allowed to submit such reports.

  • Should the permission requirements be carried over from the pilot?
  • Can temporary users submit emergency reports?
  • How should user blocks, including partial blocks, be considered when checking permissions?

See also T340146: Prevent temporary users from being able to submit incident reports

Event Timeline

kostajh added subscribers: Madalina, kostajh.

@Madalina assigning to you to clarify the answers here.

  • Yes, permission requirements should be carried over from the pilot
  • Temporary users and blocked users (including users with partial blocks) cannot submit reports

I think the remaining work here then is to ensure that users who have partial blocks cannot submit reports. (I expect that communities will have different opinions about this, though.)

Yep, how we handle temporary users and blocked/partially blocked users will require a larger discussion. Let's stick to the current permissions for the MVP and we'll revisit once we also have some data in.

mszabo changed the task status from Open to In Progress.Nov 14 2024, 6:47 AM
mszabo claimed this task.

Change #1091170 had a related patch set uploaded (by Máté Szabó; author: Máté Szabó):

[mediawiki/extensions/ReportIncident@master] Update permissions error checking

https://gerrit.wikimedia.org/r/1091170

I think this task can skip QA, as the patch makes some small changes around e.g. showing localized error messages.

Change #1091170 merged by jenkins-bot:

[mediawiki/extensions/ReportIncident@master] Update permissions error checking

https://gerrit.wikimedia.org/r/1091170

Change #1091219 had a related patch set uploaded (by Máté Szabó; author: Máté Szabó):

[mediawiki/extensions/ReportIncident@master] dialog: Compute title based on current step

https://gerrit.wikimedia.org/r/1091219

mszabo removed a project: Patch-For-Review.