Skip to content
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

[TTAHUB-199] add all regions to activity report dropdown if user has central office #503

Merged
merged 2 commits into from
Jul 15, 2021

Conversation

thewatermethod
Copy link
Collaborator

Description of change

Added "All Regions" to the region select dropdown on the activity-reports page if the user has "central office" as their home region.

How to test

Navigate to the landing page as a user with central office (14) as their home region. Notice that "All regions" is present in the dropdown. Switch between regions. The alerts and reports should change for that region. When All Regions are selected, the new report button will not appear.

Issue(s)

Checklists

Every PR

  • Meets issue criteria
  • JIRA ticket status updated
  • Code is meaningfully tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • [n/a] API Documentation updated
  • [n/a] Boundary diagram updated
  • [n/a] Logical Data Model updated
  • [n/a] Architectural Decision Records written for major infrastructure decisions

Production Deploy

  • Staging smoke test completed

After merge/deploy

  • Update JIRA ticket status

#357)

* add all regions to activity report dropdown if user has central office

* backend should return data for all widgets

* backend respect scope of region: 14

* update backend unit tests

* update frontend tests

* revert overview widget (almost)

* reduce min width

* deploy ar selector to sandbox

* fix csv downloads

* fix broken unit test

* good news, fewer known issues
@PatricePascual-ACF PatricePascual-ACF merged commit 0013a32 into HHS:main Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants