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

[1/1][docs] Add FCM V1 instructions to Google Service Account Key docs #146

Conversation

christopherwalter
Copy link
Contributor

Why

We need to amend these docs to support using a Google Service Account Key for sending Android notifications via FCM V1. These docs are linked in the CLI when users are configuring Google Service Account Keys for submissions and notifications. (See @quinlanj's comment on this PR in eas-cli: expo/eas-cli#2197 (comment)).

How

  1. Update the top-level intro with an explanation of the two use cases for uploading a Google Service Account Key to EAS.
  2. Move existing instructions for eas submit into a new subsection
  3. Add a subsection for instructions for uploading GSAK for FCM V1

Test Plan

Get feedback on copy, check that preview markdown looks good

Copy link

linear bot commented Feb 7, 2024

@christopherwalter christopherwalter force-pushed the chriswalter/eng-11172-fcmv1-specify-creds-in-eas-cli-2-gsak-docs branch from 5336485 to 63b13ca Compare February 7, 2024 14:56
@christopherwalter christopherwalter marked this pull request as ready for review February 7, 2024 14:57
@christopherwalter
Copy link
Contributor Author

@quinlanj @amandeepmittal this is a pretty rough draft but I was hoping to get your feedback on the copy and layout changes. I'll update with the correct image URLs before landing.

creating-google-service-account.md Outdated Show resolved Hide resolved
creating-google-service-account.md Outdated Show resolved Hide resolved
@christopherwalter christopherwalter force-pushed the chriswalter/eng-11172-fcmv1-specify-creds-in-eas-cli-2-gsak-docs branch 2 times, most recently from 648ddf9 to e0d3854 Compare February 8, 2024 18:09
# Why
We need to amend these docs to support using a Google Service Account Key for sending Android notifications via FCM V1. These docs are linked in the CLI when users are configuring Google Service Account Keys for submissions and notifications. (See @quinlanj's comment on this PR in eas-cli: expo/eas-cli#2197 (comment)).

# How
1. Update the top-level intro with an explanation of the two use cases for uploading a Google Service Account Key to EAS.
2. Move existing instructions for `eas submit` into a new subsection
3. Add a subsection for instructions for uploading GSAK for FCM V1

# Test Plan
Get feedback on copy, check that preview markdown looks good
@christopherwalter christopherwalter force-pushed the chriswalter/eng-11172-fcmv1-specify-creds-in-eas-cli-2-gsak-docs branch from e0d3854 to 6302597 Compare February 8, 2024 23:58
Copy link
Member

@quinlanj quinlanj left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Member

@amandeepmittal amandeepmittal left a comment

Choose a reason for hiding this comment

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

Lgtm 👍 Thanks for updating this.

Left one minor suggestion about the callout and being consistent when addressing use cases.

creating-google-service-account.md Outdated Show resolved Hide resolved
@christopherwalter christopherwalter merged commit a815f84 into main Feb 9, 2024
@christopherwalter christopherwalter deleted the chriswalter/eng-11172-fcmv1-specify-creds-in-eas-cli-2-gsak-docs branch February 9, 2024 22:18
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.

3 participants