-
Notifications
You must be signed in to change notification settings - Fork 67
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
Update README.md #2306
Update README.md #2306
Conversation
Added Amplify Logo
Added Amplify Logo in the file. |
@sobolk and @rtpascual ; can you please review this request? |
@josefaidt is this fine? |
Hey @PawanMude thank you for taking the time to file this! Do you mind if we pivot to using one of the logo assets from the Amplify documentation site? There are a few different sizes available You can see the other sizes here https://github.com/aws-amplify/docs/blob/main/public/assets/icon/favicon-purple-64x64.png |
Updated Update README.md with Amplify Doc link Co-authored-by: Kamil Sobol <[email protected]>
|
@josefaidt : I've been using AWS Amplify pretty much since day one of it's launch, and the orange logo just fits the AWS vibe. The blue one, though, gives off the feeling that ; someone at AWS is missing Azure :) Logo is like a national flag for the community - we don’t keep changing the colors just because there’s a shift in leadership across the legislative, executive, and judicial branches. Sometimes we need to agree to disagree; so if change is the flavor, and if you still want me to update the logo, I'm fine going with the community. |
Updated new Logo
@josefaidt can you please review updates? |
Co-authored-by: josef <[email protected]>
Added Amplify Logo
Problem
Issue number, if available:
Changes
Corresponding docs PR, if applicable:
Validation
Checklist
run-e2e
label set.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.