Skip to content

Conversation

@sgratzl
Copy link
Member

@sgratzl sgratzl commented Mar 30, 2021

@netlify
Copy link

netlify bot commented Mar 30, 2021

Deploy preview for cmu-delphi-main ready!

Built with commit 11a4cb1

https://deploy-preview-292--cmu-delphi-main.netlify.app

@sgratzl sgratzl self-assigned this Apr 5, 2021
@RoniRos
Copy link
Member

RoniRos commented Apr 19, 2021

Thanks @sgratzl !
I'm sorry but I just noticed one text issue:
In the very last line of the page, we should use the full name of the CDC, which is:
U.S. Centers for Disease Control and Prevention (US CDC)

Please note the plural form ("Centers").
We prepend "U.S." because nowadays there are others, e.g. E-CDC in Europe, Chinese CDC, and others.

@sgratzl
Copy link
Member Author

sgratzl commented Apr 19, 2021

Thanks @sgratzl !
I'm sorry but I just noticed one text issue:
In the very last line of the page, we should use the full name of the CDC, which is:
U.S. Centers for Disease Control and Prevention (US CDC)

Please note the plural form ("Centers").
We prepend "U.S." because nowadays there are others, e.g. E-CDC in Europe, Chinese CDC, and others.

image

@Raphael-Hyde
Copy link
Contributor

LGTM :)

Thanks @sgratzl

@sgratzl sgratzl requested a review from chinandrew April 20, 2021 11:52
@chinandrew
Copy link
Contributor

Is the extra whitespace between "for" and "Disease" so the left and right edges are justified?
Screenshot from 2021-04-20 09-40-49

@sgratzl
Copy link
Member Author

sgratzl commented Apr 20, 2021

Is the extra whitespace between "for" and "Disease" so the left and right edges are justified?

I didn't create this image but just used the one from a a previous website version. But I guess that was the motivation

Copy link
Contributor

@chinandrew chinandrew left a comment

Choose a reason for hiding this comment

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

:shipit: :shipit: :shipit:

@sgratzl sgratzl merged commit 756f877 into dev Apr 20, 2021
@sgratzl sgratzl deleted the sgratzl/center-of-excellence branch April 20, 2021 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants