Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Anonymous Ticket Portal Anonymous Ticket Portal
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 27
    • Issues 27
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • The Tor Project
  • TPA
  • Anonymous Ticket PortalAnonymous Ticket Portal
  • Merge requests
  • !60

Added Mod Functionality for GitlabAccountRequests

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged MariaV requested to merge gl_account_requests_mod into master Jan 25, 2021
  • Overview 0
  • Commits 19
  • Pipelines 0
  • Changes 8

Created the urls, views, and templates related to Gitlab Account Requests for moderators, as did not want gitlab account request functionality to exist for users until moderators were able to test/approve items. This merge request includes:

  • Added a separate variable in settings.py for GitlabAccountRequest secret tokens, as tokens for account request may be at a different permission level than issues/notes.
  • moved formset calls to within the 'moderator' view function (versus at global level in views.py) to make it easier to traceback to where they're coming from down the line.
  • Added comments to forms.py template re: formsets for moderator views
  • Added (and then removed) an optional foreign-key to useridentifier to the GitlabAccountRequest model--removed as there are some questions re: handling delete, etc, and wanted to clarify - will likely need to move into a separate merge request.
  • Added an update view for GitlabAccountRequest for moderators to allow for updates to pending account requests.
  • Added url and template for GitlabAccountRequest update view and added links to view on 'moderators' template.
  • Added gitlab account request formset to forms.py and logic to 'moderator' view, including for get and post, and tested using account with "Moderators" group or "Account Approvers" group assigned.
  • Simplified tests for "moderator" and "account approver" to make them clearer and easier to follow/implement in view logic.
  • Added some tests in tests.py for GET/POST on new views. More tests pending in a separate merge request.
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: gl_account_requests_mod