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.

Sign up for GitHub

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

Jump to bottom

Remove Any proto decoding from firestore #1371

Merged

colerogers merged 1 commit into master from colerogers.change-firestore-decoding

Mar 29, 2023

Contributor

colerogers commented Mar 29, 2023 •

edited

Yank the decoding to the Any proto to match Eventarc encoding

const dataBuffer = Buffer.from(data);
const firestoreDecoded = DocumentEventData.decode(dataBuffer);
...


          remove Any proto decoding

463c3a9

colerogers requested review from marcgovi, TheIronDev, taeold and blidd-google

March 29, 2023 12:24

marcgovi approved these changes

View reviewed changes

TheIronDev approved these changes

View reviewed changes

colerogers merged commit 74a5553 into master

15 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment