e64594ac44
Resolve "`Member.add_user`doesn't detect existing members that have requested access" ## What does this MR do? This merge request handle the case when an access requester is added to a group or project (via the members page or the API). In `Member.add_user`, if an access requester already exists, we simply accept their request (and set the `created_by`, `access_level` and `expires_at` attributes if given). ## Are there points in the code the reviewer needs to double check? I've taken the opportunity to cleanup the whole `{Group,Project}Member.add_user*` methods since it was quite a mess. ## What are the relevant issue numbers? Closes #21983 See merge request !6393 |
||
---|---|---|
.. | ||
admin | ||
ci | ||
groups | ||
import | ||
oauth | ||
profiles | ||
projects | ||
abuse_reports_controller_spec.rb | ||
application_controller_spec.rb | ||
autocomplete_controller_spec.rb | ||
blob_controller_spec.rb | ||
groups_controller_spec.rb | ||
health_check_controller_spec.rb | ||
help_controller_spec.rb | ||
invites_controller_spec.rb | ||
namespaces_controller_spec.rb | ||
notification_settings_controller_spec.rb | ||
projects_controller_spec.rb | ||
registrations_controller_spec.rb | ||
root_controller_spec.rb | ||
sent_notifications_controller_spec.rb | ||
sessions_controller_spec.rb | ||
snippets_controller_spec.rb | ||
uploads_controller_spec.rb | ||
users_controller_spec.rb |