这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@dackers86
Copy link
Member

@dackers86 dackers86 commented May 5, 2021

Checklist

Testing

Jest Tests

N/a for this module.

Updated engine

image

Successful Installation

image

Successful result following installation

image

image

@google-cla google-cla bot added the cla: yes Author has signed the CLA label May 5, 2021
@codecov-commenter
Copy link

codecov-commenter commented May 5, 2021

Codecov Report

Merging #657 (e8cc1f8) into next (612cbbf) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             next     #657   +/-   ##
=======================================
  Coverage   50.65%   50.65%           
=======================================
  Files          30       30           
  Lines        1313     1313           
  Branches      193      193           
=======================================
  Hits          665      665           
  Misses        647      647           
  Partials        1        1           

@dackers86
Copy link
Member Author

Testing

image

image

image

- This extension uses other Firebase and Google Cloud Platform services, which have associated charges if you exceed the service’s free tier:
- Cloud Storage
- Cloud Functions (Node.js 10+ runtime. [See FAQs](https://firebase.google.com/support/faq#expandable-24))
- Cloud Functions (Node.js 12+ runtime. [See FAQs](https://firebase.google.com/support/faq#expandable-24))

Choose a reason for hiding this comment

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

I was curious so I checked. New URL: https://firebase.google.com/support/faq#extensions-pricing

Copy link
Member Author

Choose a reason for hiding this comment

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

Great spot Mike, reading that article suggest that this change probably isn't needed as part of the PR? As the document specifically notes v10+..?

Could always update that link as a separate PR?

Choose a reason for hiding this comment

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

Mmm good point @dackers86 - probably leave 10+ runtime in the comment (since that is the version in the pricing document) but definitely point to the new URL

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated.

@Salakar Salakar marked this pull request as ready for review May 11, 2021 16:28
@Salakar Salakar requested a review from a team as a code owner May 11, 2021 16:28
Co-authored-by: Jeff <jhuleatt@google.com>
@dackers86 dackers86 mentioned this pull request May 12, 2021
8 tasks
@jhuleatt jhuleatt changed the title chore(storage-resize-images): migration to Node.js v12 chore(storage-resize-images): migration to Node.js v14 Jun 15, 2021
@jhuleatt jhuleatt added this to the Jun-21-2 milestone Jun 15, 2021
@pavelgj pavelgj merged commit a28f25c into next Jun 16, 2021
pavelgj added a commit that referenced this pull request Jun 17, 2021
* chore(delete-user-data): refactor test data into delete user data (#655)

* feat(*): added warsaw (europe-central2) location (#677)

Co-authored-by: Rosário P. Fernandes <rosariofernandes51@gmail.com>

* chore(storage-resize-images): migration to Node.js v14 (#657)

* chore(storage-resize-images): migration to Node.js v12

* Undo readme, update preinstall

* Documentation and package updates

* Update storage-resize-images/README.md

Co-authored-by: Jeff <jhuleatt@google.com>

* node 12 to 14

Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com>
Co-authored-by: Jeff <jhuleatt@google.com>

* chore(delete-user-data): migration to Node.js v14 (#661)

* chore(delete-user-data): migration to Node.js v12

* Updated engine to node14, build fixes and updated target lib

* chore(firestore-counter): migration to Node.js v14 (#664)

* chore(firestore-counter): migration to Node.js v12

* Updated engine to node14, build fixes and updated target lib

* chore(firestore-send-email): migration to Node.js v14 (#665)

* chore(firestore-send-email): migration to Node.js v12

* Updated engine to node14, build fixes and updated target lib

* chore(firestore-shorten-urls-bitly): migration to Node.js v14 (#667)

* chore(firestore-send-email): migration to Node.js v12

* chore(firestore-shortern-urls-bitly): migration to Node.js v12

* Updated engine to node14, build fixes and updated target lib

* reverted send email updates

* reverted additional send email updates

* chore(firestore-translate-text): migration to Node.js v14 (#668)

* chore(firestore-translate-text): migration to Node.js v12

* Updated engine to node14, build fixes and updated target lib

* chore(rtdb-limit-child-nodes): migration to Node.js v14 (#669)

* chore(rtdb-limit-child-nodes-node12): migration to Node.js v12

* Updated engine to run node14

* Updated target mapping and build fixes

* chore(firestore-bigquery-export): migration to Node.js v14 (#662)

* chore(firestore-bigquery-export): migration to Node.js v12

* Reverted change tracker updates, updated export dependencies

* Updated engine to node14, build fixes and updated target lib

* Update firestore-bigquery-export/functions/tsconfig.json

Co-authored-by: Jeff <jhuleatt@google.com>
Co-authored-by: Pavel Jbanov <pavelgj@gmail.com>

* chore(*): regenerate release notes for June 21 2021 release (#680)

Co-authored-by: Darren Ackers <ackers86@hotmail.com>
Co-authored-by: Rosário P. Fernandes <rosariofernandes51@gmail.com>
Co-authored-by: Mike Diarmid <mike.diarmid@gmail.com>
Co-authored-by: Jeff <jhuleatt@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Author has signed the CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants