Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

docs: update BulkWriterError documentation#1518

Merged
thebrianchen merged 1 commit intomasterfrom
bc/bulk-docs2
May 26, 2021
Merged

docs: update BulkWriterError documentation#1518
thebrianchen merged 1 commit intomasterfrom
bc/bulk-docs2

Conversation

@thebrianchen
Copy link
Copy Markdown

I realized that our current documentation does not have any property definitions for BulkWriterError.

@thebrianchen thebrianchen requested review from a team May 26, 2021 18:20
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label May 26, 2021
@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/nodejs-firestore API. label May 26, 2021
Comment thread dev/src/index.ts
* @property {string} message The error message of the error.
* @property {DocumentReference} documentRef The document reference the operation was
* performed on.
* @property {'create' | 'set' | 'update' | 'delete'} operationType The type
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I assume you verified that this looks good in the generated docs.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yep!

@thebrianchen thebrianchen merged commit 0515d1f into master May 26, 2021
@thebrianchen thebrianchen deleted the bc/bulk-docs2 branch May 26, 2021 19:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: firestore Issues related to the googleapis/nodejs-firestore API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants