Skip to content

Commit

Permalink
updating schedule functions. now they are running on europe-west1
Browse files Browse the repository at this point in the history
  • Loading branch information
Alona-Kirichenko committed Aug 27, 2024
1 parent 6f77523 commit 0460533
Show file tree
Hide file tree
Showing 9 changed files with 149 additions and 144 deletions.
8 changes: 7 additions & 1 deletion .firebaserc
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,11 @@
"default": "do-good-get-good-2f6cc",
"dev": "dev-do-good-get-good"
},
"targets": {}
"targets": {},
"etags": {
"dev-do-good-get-good": {
"extensionInstances": {}
}
},
"dataconnectEmulatorConfig": {}
}
2 changes: 1 addition & 1 deletion functions/.gitignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
node_modules/
ServiceAccount/
functions/firebase-debug.log
.env*
.env*
2 changes: 0 additions & 2 deletions functions/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,3 @@ here is more info https://github.com/invertase/react-native-firebase/issues/6622

Most of second gen functions is at europe-north1.
But onSchrdul function doesn't support all region https://github.com/firebase/firebase-functions/issues/1293#issuecomment-1297524616
In summer 2024 I tried europe-north1, europe-west1, europe-west2 it didn't work, so I deployed it to default us-central1. But maybe in future they will fix it.
So when you deploy onSchedul function, just comment awaty setGlobalOptions({ region: "europe-north1" }); at config file
214 changes: 105 additions & 109 deletions functions/package-lock.json

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions functions/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@
"date-fns": "^2.28.0",
"dotenv": "^16.4.5",
"exceljs": "^4.3.0",
"firebase-admin": "^12.1.1",
"firebase-functions": "^5.0.1"
"firebase-admin": "^12.4.0",
"firebase-functions": "^5.1.0"
},
"devDependencies": {
"firebase-functions-test": "^0.2.0"
Expand Down
5 changes: 3 additions & 2 deletions functions/src/deletePostAfterOneYearSecondGen.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,11 @@ const deleteImageInStore = async (imgPath) => {
});
};

// when you deploy onSchedul function, just comment awaty setGlobalOptions({ region: "europe-north1" }); at config file. More info at README
// onSchedule don't suport region europe-north1. More info at README

exports.deletePostAfterOneYearSecondGen = onSchedule(
"0 0 1 1,4,7,10 *",
{ schedule: "0 0 1 1,4,7,10 *", region: "europe-west1" },

async (event) => {
const oneYearAgo = new Date();
oneYearAgo.setFullYear(oneYearAgo.getFullYear() - 1);
Expand Down
25 changes: 14 additions & 11 deletions functions/src/updateMonthSecondGen.js
Original file line number Diff line number Diff line change
@@ -1,15 +1,18 @@
const { onSchedule } = require("firebase-functions/v2/scheduler");
const { getFirestore } = require("firebase-admin/firestore");

// when you deploy onSchedul function, just comment awaty setGlobalOptions({ region: "europe-north1" }); at config file. More info at README
exports.updateMonthSecondGen = onSchedule("0 0 1 * *", async (event) => {
await getFirestore()
.collection("Users")
.get()
.then((snap) => {
snap.forEach((doc) => {
doc.ref.update({ total_confirmed_hours: 0, total_hours_month: 0 });
// onSchedule don't suport region europe-north1. More info at README
exports.updateMonthSecondGen = onSchedule(
{ schedule: "0 0 1 * *", region: "europe-west1" },
async (event) => {
await getFirestore()
.collection("Users")
.get()
.then((snap) => {
snap.forEach((doc) => {
doc.ref.update({ total_confirmed_hours: 0, total_hours_month: 0 });
});
});
});
return console.log("Successful reset of time for month and approvedtimes");
});
return console.log("Successful reset of time for month and approvedtimes");
}
);
30 changes: 17 additions & 13 deletions functions/src/updateYearSecondGen.js
Original file line number Diff line number Diff line change
@@ -1,18 +1,22 @@
const { onSchedule } = require("firebase-functions/v2/scheduler");
const { getFirestore } = require("firebase-admin/firestore");

exports.updateYearSecondGen = onSchedule("0 0 1 1 *", async (event) => {
try {
await getFirestore()
.collection("Users")
.get()
.then((snap) => {
snap.forEach((doc) => {
doc.ref.update({ total_hours_year: 0 });
// onSchedule don't suport region europe-north1. More info at README
exports.updateYearSecondGen = onSchedule(
{ schedule: "0 0 1 1 *", region: "europe-west1" },
async (event) => {
try {
await getFirestore()
.collection("Users")
.get()
.then((snap) => {
snap.forEach((doc) => {
doc.ref.update({ total_hours_year: 0 });
});
});
});
return console.log("Successful reset of time for year");
} catch (error) {
console.error("Error fetching user data:", error);
return console.log("Successful reset of time for year");
} catch (error) {
console.error("Error fetching user data:", error);
}
}
});
);
3 changes: 0 additions & 3 deletions functions/ui-debug.log

This file was deleted.

0 comments on commit 0460533

Please sign in to comment.