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.

Sign up for GitHub

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

Jump to bottom

Display My Alerts #305

Merged
merged 59 commits into from
Feb 18, 2021
Merged

Display My Alerts #305

merged 59 commits into from
Feb 18, 2021

Conversation

kryswisnaskas
Copy link
Collaborator

@kryswisnaskas kryswisnaskas commented Feb 18, 2021 •
edited
Loading

108

Description of change
Add print styles

Issue(s)

181 191 192

Description of change
Display My Alerts for managers, authors and collaborators.

Issue(s)

296

Description of change
Create a context menu component

Issue(s)

Checklist

  • Meets issue criteria
  • Code tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • Documentation updated

dcloud and others added 30 commits February 5, 2021 10:08
@dcloud
Setup a '@media print' section in App.css with expected styles
bd2a13c
@dcloud
Add classes to components so they can be selected in CSS for print-re…
97e06a1 
…lated styles
@dcloud
More print style adjustments and fixes
1cbbde2
@dcloud
Add PrintSummary with reportCreator information
68d9849
@dcloud
12pt base font-size for print
e14a77f
@dcloud
Fix font-family in PrintSummary
886773c
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
7a9578f
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
d57a300
@dcloud
Add reportCreator to tests so we don't get props errors
709c954
@dcloud
Use react-hook-form's watch func to watch for textarea changes, and s…
8c8880e 
…et no-print or yes-print classes
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
3f75acf
@dcloud
Use .no-print class when possible. Still think bem-style class names …
1cb550a 
…can come in handy for other uses
@dcloud
Add label with AR status, for both print and web
7bd5679
@dcloud
Alter how name, role text is created, and expect values for both props
463e5a1
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
70de3a4
@dcloud
Fix additionalNotes textarea value watching copypasta error
791dca0
@kryswisnaskas
Merge branch 'js-145-activity-report-display-id' of https://github.co…
8f474ae 
…m/adhocteam/Head-Start-TTADP into kw-my-alerts
@kryswisnaskas
Merge branch 'main' of https://github.com/adhocteam/Head-Start-TTADP …
8cfc965 
…into kw-my-alerts
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
31ff8a3
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
e1686dd
@dcloud
Update styling of reviewItem border lines
9e8d280
@dcloud
Update useWatch to pass 'control' object from useFormContext
344cfe4
@dcloud
See if 'control' is auto provided by FormContext
bb70cc9
@dcloud
Switch from useWatch to watch from useFormContext
8cf10ac
@kryswisnaskas
Display My Alerts
59e7621
@dcloud
Try the pass-down way
44a2091
@dcloud
Revert "Try the pass-down way"
645453d 
This reverts commit 44a2091.
@dcloud
Make PrintSummary work without a reportCreator
0d18635
@dcloud
Fix tests that now need FormProvider.
de403d1
@kryswisnaskas
Fix lint errors
123f773
dcloud and others added 26 commits February 15, 2021 17:41
@dcloud
Navigator: make reportCreator not required
68e5f7a
@dcloud
Tests for PrintSummary
00911e9
@dcloud
Remove extra bottom-border
85e9467
@jasalisbury
Merge branch 'main' into js-296-add-context-menu
f04d3e5
@kryswisnaskas
Modified links to reports
f5508c7
@kryswisnaskas
Fix needs_status display
7a05073
@kryswisnaskas
Modify alerts query
49fbf96
@kryswisnaskas
Merge branch 'main' into kw-my-alerts
c24af59
@jasalisbury
Merge branch 'main' into js-296-add-context-menu
3dba4c2
@jasalisbury
Merge branch 'main' into js-296-add-context-menu
1e0487a
@jasalisbury
Add aria-haspopup to context menu
3c7e5bb
@jasalisbury
Merge branch 'main' into js-296-add-context-menu
a02c309
@dcloud
Clarify test names for PrintSummary
2aa3491
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
245e94f
@jasalisbury
Context menu closes on escape
b49274d
@kryswisnaskas
Use constants instead of hardcoding; fix props types
267041d
@kryswisnaskas
Merge branch 'kw-my-alerts' of https://github.com/adhocteam/Head-Star…
c3eb8c7 
…t-TTADP into kw-my-alerts
@kryswisnaskas
Merge branch 'main' into kw-my-alerts
09212e1
@dcloud
Refactor Review-related components to utilize useFormContext, rather …
cbfc0a0 
…than passing down props
@jasalisbury
Merge pull request #171 from adhocteam/js-296-add-context-menu
39ceb7e 
Add the context menu component
@kryswisnaskas
Merge branch 'main' into kw-my-alerts
8cca514
@dcloud
Merge branch 'main' into dcloud/print-styles-for-activity-reports
0676983
@dcloud
Merge pull request #155 from adhocteam/dcloud/print-styles-for-activi…
a82d5c0 
…ty-reports

Print styles for activity reports
@kryswisnaskas
Fix test coverage
721b6c7
@kryswisnaskas
Merge branch 'main' into kw-my-alerts
b12e8e3
@kryswisnaskas
Merge pull request #170 from adhocteam/kw-my-alerts
54cff22 
Display my alerts
rahearn
rahearn approved these changes Feb 18, 2021
View reviewed changes
Copy link
Contributor

@rahearn rahearn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@rahearn rahearn merged commit 5c70cc3 into HHS:main Feb 18, 2021
thewatermethod pushed a commit that referenced this pull request Jul 9, 2021
@kryswisnaskas
Add an ability to view a diagnostics table in the admin UI (#305)
2c717ed 
* Request errors api

* Request errors api

* Added data provider

* Added show

* Add backend tests

* Address feedback from review, add frontend test

* Deploy to sandbox
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rahearn rahearn rahearn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@kryswisnaskas @rahearn @dcloud @jasalisbury