-
-
Notifications
You must be signed in to change notification settings - Fork 141
updated firebase and firebase-admin #1625
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
attempting to update runWith with onRequest scraper.ts probably all need to be updated |
.firestore.document("/generalCourts/{court}/bills/{billId}") | ||
.onWrite(async (change, context) => { | ||
const params = context.params, | ||
// export const updateBillTracker = runWith({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove
We are still ready for the Firebase Function V1 -> V2 upgrade, but we can separate that out from the library version update that's already complete - going to close this older attempt so we can look at the problem in the new environment with fresh eyes in te |
Summary
Add a short summary of the changes, and a reference to the original issue using
#
and the issue number, like #1Checklist
Screenshots
Add some screenshots highlighting your changes.
Known issues
If you've run against limitations or caveats, include them here. Include follow-up issues as well.
Steps to test/reproduce
For each feature or bug fix, create a step by step list for how a reviewer can test it out. E.g.: