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

docs(firestore-bigquery-export): clarify meaning of "exclude old data" #2024

Open
wants to merge 5 commits into
base: next
Choose a base branch
from

Conversation

gregfenton
Copy link
Contributor

The original description is generically worded. This additional specificity should make it clear to Firestore users what is meant by "old data".

The original description is generically worded.  This additional specificity should make it clear to Firestore users what is meant by "old data".
@cabljac
Copy link
Contributor

cabljac commented Apr 23, 2024

Hi there! thanks for contributing!

The README.md isn't actually directly edited, what you need to do is find the corresponding documentation in the extension.yaml or the PREINSTALL.md and amend it there, before running npm run generate-readme in the functions directory

Can you do this and push changes, and I'll review?

@cabljac cabljac changed the title doc: clarify meaning of "exclude old data" docs(firestore-bigquery-export): clarify meaning of "exclude old data" Apr 23, 2024
@gregfenton
Copy link
Contributor Author

@cabljac change made as per your direction. Thanks!

@cabljac cabljac force-pushed the next branch 3 times, most recently from c8885bd to d1481a2 Compare April 24, 2024 10:27
@pr-Mais
Copy link
Member

pr-Mais commented May 30, 2024

Hi @gregfenton, the README is still missing, could you please npm run generate-readme and push the generated file?

@gregfenton
Copy link
Contributor Author

Is there a document describing how to make docs PRs in a "clean way" ? I'd prefer to get it right on the first pass, and not bother busy people like y'all 😊

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.

None yet

3 participants