Merge branch 'dev' into qa
All checks were successful
Deploy FitLien services to QA / Deploy to QA (push) Successful in 3m37s
All checks were successful
Deploy FitLien services to QA / Deploy to QA (push) Successful in 3m37s
This commit is contained in:
commit
209c7c65b0
@ -46,10 +46,11 @@ interface PersonalTrainerAssign {
|
||||
|
||||
export const checkExpiredMemberships = onSchedule(
|
||||
{
|
||||
schedule: "*/5 * * * *",
|
||||
schedule: "0 8,14,20 * * *",
|
||||
timeZone: "UTC",
|
||||
region: "#{SERVICES_RGN}#",
|
||||
},
|
||||
|
||||
async (event) => {
|
||||
logger.info("Starting scheduled membership expiry check...");
|
||||
|
||||
@ -58,10 +59,17 @@ export const checkExpiredMemberships = onSchedule(
|
||||
const expiredMemberships = await findExpiredMemberships();
|
||||
const expiringMemberships = await findMembershipsExpiringIn10Days();
|
||||
|
||||
const expiredMembershipsWithoutExpiryDate = await findExpiredMembershipsWithoutExpiryDate();
|
||||
const expiredMembershipsWithoutExpiryDate =
|
||||
await findExpiredMembershipsWithoutExpiryDate();
|
||||
|
||||
if (expiredMemberships.length === 0 && expiringMemberships.length === 0 && expiredMembershipsWithoutExpiryDate.length === 0) {
|
||||
logger.info("No expired, expiring, or unprocessed expired memberships found.");
|
||||
if (
|
||||
expiredMemberships.length === 0 &&
|
||||
expiringMemberships.length === 0 &&
|
||||
expiredMembershipsWithoutExpiryDate.length === 0
|
||||
) {
|
||||
logger.info(
|
||||
"No expired, expiring, or unprocessed expired memberships found."
|
||||
);
|
||||
return;
|
||||
}
|
||||
|
||||
@ -78,15 +86,29 @@ export const checkExpiredMemberships = onSchedule(
|
||||
);
|
||||
|
||||
const updateResults = await Promise.allSettled(
|
||||
expiredMembershipsWithoutExpiryDate.map((m) => updateExpiryDateForExpiredMembership(m.id, m.data))
|
||||
expiredMembershipsWithoutExpiryDate.map((m) =>
|
||||
updateExpiryDateForExpiredMembership(m.id, m.data)
|
||||
)
|
||||
);
|
||||
|
||||
const expiredSuccessful = expiredResults.filter(r => r.status === "fulfilled").length;
|
||||
const expiredFailed = expiredResults.filter(r => r.status === "rejected").length;
|
||||
const expiringSuccessful = expiringResults.filter(r => r.status === "fulfilled").length;
|
||||
const expiringFailed = expiringResults.filter(r => r.status === "rejected").length;
|
||||
const updateSuccessful = updateResults.filter(r => r.status === "fulfilled").length;
|
||||
const updateFailed = updateResults.filter(r => r.status === "rejected").length;
|
||||
const expiredSuccessful = expiredResults.filter(
|
||||
(r) => r.status === "fulfilled"
|
||||
).length;
|
||||
const expiredFailed = expiredResults.filter(
|
||||
(r) => r.status === "rejected"
|
||||
).length;
|
||||
const expiringSuccessful = expiringResults.filter(
|
||||
(r) => r.status === "fulfilled"
|
||||
).length;
|
||||
const expiringFailed = expiringResults.filter(
|
||||
(r) => r.status === "rejected"
|
||||
).length;
|
||||
const updateSuccessful = updateResults.filter(
|
||||
(r) => r.status === "fulfilled"
|
||||
).length;
|
||||
const updateFailed = updateResults.filter(
|
||||
(r) => r.status === "rejected"
|
||||
).length;
|
||||
|
||||
logger.info(
|
||||
`Completed processing. Expired - Success: ${expiredSuccessful}, Failed: ${expiredFailed}. Expiring - Success: ${expiringSuccessful}, Failed: ${expiringFailed}. Updates - Success: ${updateSuccessful}, Failed: ${updateFailed}`
|
||||
@ -97,9 +119,6 @@ export const checkExpiredMemberships = onSchedule(
|
||||
}
|
||||
);
|
||||
|
||||
|
||||
|
||||
|
||||
async function findExpiredMembershipsWithoutExpiryDate(): Promise<
|
||||
Array<{ id: string; data: MembershipData }>
|
||||
> {
|
||||
@ -110,7 +129,10 @@ async function findExpiredMembershipsWithoutExpiryDate(): Promise<
|
||||
.where("status", "==", "EXPIRED")
|
||||
.get();
|
||||
|
||||
const membershipsWithoutExpiryDate: Array<{ id: string; data: MembershipData }> = [];
|
||||
const membershipsWithoutExpiryDate: Array<{
|
||||
id: string;
|
||||
data: MembershipData;
|
||||
}> = [];
|
||||
|
||||
snapshot.docs.forEach((doc) => {
|
||||
const data = doc.data() as MembershipData;
|
||||
@ -121,7 +143,10 @@ async function findExpiredMembershipsWithoutExpiryDate(): Promise<
|
||||
|
||||
return membershipsWithoutExpiryDate;
|
||||
} catch (error) {
|
||||
logger.error("Error finding expired memberships without expiry date:", error);
|
||||
logger.error(
|
||||
"Error finding expired memberships without expiry date:",
|
||||
error
|
||||
);
|
||||
throw error;
|
||||
}
|
||||
}
|
||||
@ -130,7 +155,10 @@ async function updateExpiryDateForExpiredMembership(
|
||||
membershipData: MembershipData
|
||||
): Promise<void> {
|
||||
try {
|
||||
if (!membershipData.subscription || !membershipData.subscription.frequency) {
|
||||
if (
|
||||
!membershipData.subscription ||
|
||||
!membershipData.subscription.frequency
|
||||
) {
|
||||
logger.warn(`Skipping membership ${membershipId} - no subscription data`);
|
||||
return;
|
||||
}
|
||||
@ -156,9 +184,14 @@ async function updateExpiryDateForExpiredMembership(
|
||||
updatedAt: admin.firestore.FieldValue.serverTimestamp(),
|
||||
});
|
||||
|
||||
logger.info(`Updated expiry date for expired membership ${membershipId}: ${expiryDate.toISOString()}`);
|
||||
logger.info(
|
||||
`Updated expiry date for expired membership ${membershipId}: ${expiryDate.toISOString()}`
|
||||
);
|
||||
} catch (error) {
|
||||
logger.error(`Error updating expiry date for membership ${membershipId}:`, error);
|
||||
logger.error(
|
||||
`Error updating expiry date for membership ${membershipId}:`,
|
||||
error
|
||||
);
|
||||
throw error;
|
||||
}
|
||||
}
|
||||
|
||||
@ -82,15 +82,6 @@ async function getUserAndFCMToken(
|
||||
if (notification.recipientId) {
|
||||
targetUserId = notification.recipientId;
|
||||
logger.info(`Using top-level recipientId: ${targetUserId}`);
|
||||
} else if (notification.data?.userId) {
|
||||
targetUserId = notification.data.userId;
|
||||
logger.info(`Using data.userId: ${targetUserId}`);
|
||||
} else if (notification.data?.clientId) {
|
||||
targetUserId = notification.data.clientId;
|
||||
logger.info(`Using data.clientId: ${targetUserId}`);
|
||||
} else if (notification.data?.invitorId) {
|
||||
targetUserId = notification.data.invitorId;
|
||||
logger.info(`Using data.invitorId: ${targetUserId}`);
|
||||
} else if (notification.data?.phoneNumber) {
|
||||
logger.info(
|
||||
`Looking up user by phone number from data: ${notification.data.phoneNumber}`
|
||||
|
||||
Loading…
Reference in New Issue
Block a user