Closed josueetcom closed 4 years ago
Hey all, I'd like to get some feedback on these changes. I plan to walk through them entirely for a project to make sure there aren't any gaps before marking this PR ready for review.
Additional question: Steps 4 and 5 could be written more succinctly. E.g.
Instead of:
- Configure your OAuth 2.0 Client ID to be used by your Custom Domain.
- In the GCP console, navigate to APIs & Services -> Credentials.
- Click on the OAuth 2.0 Client ID that was auto created.
- $PROJECT_ID.firebaseapp.com should appear under Authorized JavaScript origins.
- Take note of the Client ID and Client secret. You'll use them in the next step.
- Under Authorized JavaScript origins, add your custom domain.
- Click Save.
- Add Google as an Identity Provider in Identity Platform:
- In the GCP console, navigate to Identity Platform -> Providers.
- Click Add a provider.
- Select Google from the list.
- Fill in the Web Client ID and Web Client Secret fields with those from the OAuth 2.0 Client ID created in the previous step.
- Click Save.
It could be more succinctly:
- On the APIs & Services -> Credentials page, click the OAuth 2.0 Client ID that was auto created and add your custom domain under Authorized JavaScript origins.
- $PROJECT_ID.firebaseapp.com should appear under Authorized JavaScript origins.
- Take note of the Client ID and Client secret. You'll use them in the next step.
- On the Identity Platform -> Providers page, add Google as an Identity Provider.
- Fill in the Web Client ID and Web Client Secret fields with the Client ID and Client secret values from the previous step.
Which style do you all think is better?
Which style do you all think is better?
I think having more details instructions is better in this case -- one of the big pieces of feedback we've received is that our more succinct descriptions left just enough room for confusion, so I'd rather err on the side of being too wordy.
Or we can have succinct description with screenshot and highlights, like
Hmm, using a screenshot would still require instructions, at least in the screenshot you gave for steps 2 and 3.
The thing that's also tricky about detailed steps/screenshots is that if the Cloud Console UI changes then they'll quickly go out of date. I suppose this is also true of using gcloud alpha
commands though.
I'm indifferent either way. Let me know if you strongly prefer one or the other
I don't have strong preference either. As a reader I would definitively prefer more pictures. But the burden of maintenance is real. I think we should just go with the detailed version for now, and see how it works out in bug bash.
Yeah let's see how it goes in the bug bash. Maybe we can ask if screenshots would be clearer
Summary:
I also included an image of what the Firestore rules page should look like and moved the scripts to a scripts/ directory.
README.md preview: https://github.com/GoogleCloudPlatform/cloud-run-anthos-reference-web-app/blob/readme-tweaks/README.md