-
Notifications
You must be signed in to change notification settings - Fork 45
Conversation
beek seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of the comments maybe elaborated on in a separate meeting.
packages/app/src/components/weekly-summary/topical-weekly-summary-tile.tsx
Outdated
Show resolved
Hide resolved
packages/app/src/domain/topical/components/topical-theme-header.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see my comments. I also notice some merge conflicts so those need to be fixed too.
packages/app/src/components/weekly-summary/topical-weekly-summary-tile.tsx
Outdated
Show resolved
Hide resolved
packages/app/src/domain/topical/components/topical-theme-header.tsx
Outdated
Show resolved
Hide resolved
packages/app/src/components/weekly-summary/topical-weekly-summary-tile.tsx
Show resolved
Hide resolved
packages/app/src/domain/topical/components/topical-theme-header.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome job!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
Archived the coronadashboard thermometer to it's separate page