Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[user_accounts] Remove DDC restriction for pending accounts in dashboard #7054

Conversation

CamilleBeau
Copy link
Contributor

Brief summary of changes

This PR removes the restriction that DDC users are not included in pending account approval in the dashboard widget.

Testing instructions (if applicable)

  1. Request an account and choose Data Coordinating Center as site
  2. Ensure that the number of Pending account approvals increases accordingly in the My tasks section of the dashboard
  3. Ensure that when clicking on Pending account approvals, the user is included in the results
  4. Repeat steps 2-3 after creating a new user from the user_accounts module with Pending approval set to Yes.

Link(s) to related issue(s)

@CamilleBeau CamilleBeau added this to the 24.0.0 milestone Sep 22, 2020
@maltheism maltheism added the Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) label Sep 28, 2020
@driusan driusan merged commit 3ca46d8 into aces:main Oct 5, 2020
spell00 pushed a commit to spell00/Loris that referenced this pull request Nov 4, 2020
…ard (aces#7054)

Remove the restriction that DDC users are not included in pending account approval in the dashboard widget.

    Resolves aces#6962
AlexandraLivadas pushed a commit to AlexandraLivadas/Loris that referenced this pull request Jun 29, 2021
…ard (aces#7054)

Remove the restriction that DDC users are not included in pending account approval in the dashboard widget.

    Resolves aces#6962
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[login] pending accounts at DCC do not show on the dashboard (they should)
4 participants