chore: workflows hono app #1093
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Deploying openstatus-docs with
|
| Latest commit: |
d0474a5
|
| Status: | 🚫 Build failed. |
Copilot
AI
left a comment
Copilot
AI
left a comment
There was a problem hiding this comment.
Copilot reviewed 21 out of 36 changed files in this pull request and generated 1 suggestion.
Files not reviewed (15)
- apps/workflows/.dockerignore: Language not supported
- apps/workflows/.gitignore: Language not supported
- apps/workflows/Dockerfile: Language not supported
- apps/workflows/package.json: Language not supported
- apps/workflows/tsconfig.json: Language not supported
- apps/server/src/v1/statusReports/update/post.ts: Evaluated as low risk
- apps/server/src/v1/statusReports/post.ts: Evaluated as low risk
- apps/web/src/lib/auth/index.ts: Evaluated as low risk
- apps/web/src/app/(content)/blog/[slug]/page.tsx: Evaluated as low risk
- apps/server/src/v1/statusReportUpdates/post.ts: Evaluated as low risk
- apps/server/src/v1/pageSubscribers/post.ts: Evaluated as low risk
- apps/workflows/src/cron/checker.ts: Evaluated as low risk
- apps/workflows/README.md: Evaluated as low risk
- apps/workflows/src/cron/tinybird.ts: Evaluated as low risk
- apps/workflows/src/env.ts: Evaluated as low risk
apps/workflows/src/cron/emails.ts
Outdated
|
|
||
| for (const user of users) { | ||
| if (user.email) { | ||
| await email.sendFollowUp({ to: "[email protected]" }); |
There was a problem hiding this comment.
The email is being sent to a hardcoded address. It should be sent to the user's email address.
| await email.sendFollowUp({ to: "[email protected]" }); | |
| await email.sendFollowUp({ to: user.email }); |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
There was a problem hiding this comment.
@mxkaske name and shame by copilot 🤣
| import readline from "node:readline"; | ||
|
|
||
| // Function to prompt user for confirmation | ||
| const askConfirmation = async (question: string): Promise<boolean> => { |
There was a problem hiding this comment.
This is to avoid doing any accidental deletion. We ask for if the DATABASE_URL is correct and that we fetch the user data from that one.
There was a problem hiding this comment.
Seems like it's the script to delete old data?
Can't we move it to a directory script or something?
There was a problem hiding this comment.
We could run that on cron e.g.! Let's keep it there for now and consider creating a @openstatus/scripts package?
| import { Resend } from "resend"; | ||
| import { FollowUpEmail } from "../emails/followup"; | ||
|
|
||
| export class EmailClient { |
There was a problem hiding this comment.
OOP when most of our codebase is functional 🤔
No description provided.