Refactor member view by using presenter
requested to merge tmlee/gitlab-ce-28004-consider-refactoring-member-view-by-using-presenter into master
Originally opened at !9645 (closed) by @tmlee.
EE MR: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/3620
What does this MR do?
Refactor member view by using presenter:
- Create MemberPresenter alongside with GroupMemberPresenter and ProjectMemberPresenter
- Make Member model Presentable
- Move action_member_permission from MembersHelper into the MemberPresenter
- Added rspec using double, separate specs for GroupMemberPresenter and ProjectMemberPresenter
Are there points in the code the reviewer needs to double check?
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Tests added for this feature/bug - Review
-
Has been reviewed by Backend
-
-
Conform by the style guides -
Squashed related commits together
What are the relevant issue numbers?
Fixes #28004 (closed).
Edited by Rémy Coutable