-
Notifications
You must be signed in to change notification settings - Fork 26
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.
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
Allow to edit own delegation via user.update_self #2632
Allow to edit own delegation via user.update_self #2632
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMAW.
If this will not lead to widespread changes you could overhaul the grammar of the existing error messages.
@@ -14,5 +16,9 @@ Updates the request user. Removes starting and trailing spaces from `username`. | |||
|
|||
The given `gender` must be present in `organization/genders`. | |||
|
|||
`meeting_id` is only for editing meeting-internal data, and the value will be thrown away afterwards. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about:
meeting_id
is only used for processing meeting-internal data and will be omitted otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The purpose of my description was to point out that meeting_id
won't be written into the data anywhere and is just additional information given to aid action-internal calculations.
Your proposition doesn't do that, so I won't change it-
Closes #2503