Adds unit tests for the email campaign service’s validation and lifecycle rules.#358
Merged
Conversation
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…eries Agent-Logs-Url: https://github.com/swamphacks/core/sessions/1e2e08e9-0737-47dd-8a01-bddead1ad570 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com>
…tory Agent-Logs-Url: https://github.com/swamphacks/core/sessions/5c4475ce-1d02-429a-80bf-15b48fefcd07 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…um array handling
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
hieunguyent12
approved these changes
Jun 21, 2026
hieunguyent12
added a commit
that referenced
this pull request
Jun 22, 2026
* feat: implement EmailCampaignRepository with CRUD operations (#351) * feat: add the existing email routes and handler to the API * feat: add email campaigns migration with status, format, and recipient types * feat: add EmailCampaign model with status, format, and recipient types * feat: update SQLC version to v1.30.0 * feat: add SQL queries for email campaigns management * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: use sqlc.narg for nullable scheduled_at and sent_at in update queries Agent-Logs-Url: https://github.com/swamphacks/core/sessions/1e2e08e9-0737-47dd-8a01-bddead1ad570 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * style: group stdlib and third-party imports in email_campaigns repository Agent-Logs-Url: https://github.com/swamphacks/core/sessions/5c4475ce-1d02-429a-80bf-15b48fefcd07 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * init front page * started working on info modal and other styling tweaks * teams tab and more changes * add Sponsors and Partners components with modal functionality * Adds the email campaign service layer inside the existing email domain (#352) * feat: add the existing email routes and handler to the API * feat: add email campaigns migration with status, format, and recipient types * feat: add EmailCampaign model with status, format, and recipient types * feat: update SQLC version to v1.30.0 * feat: add SQL queries for email campaigns management * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: use sqlc.narg for nullable scheduled_at and sent_at in update queries Agent-Logs-Url: https://github.com/swamphacks/core/sessions/1e2e08e9-0737-47dd-8a01-bddead1ad570 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * style: group stdlib and third-party imports in email_campaigns repository Agent-Logs-Url: https://github.com/swamphacks/core/sessions/5c4475ce-1d02-429a-80bf-15b48fefcd07 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * feat: implement EmailCampaignService with CRUD operations and validation --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * feat: add email campaign HTTP handlers (#353) * feat: add the existing email routes and handler to the API * feat: add email campaigns migration with status, format, and recipient types * feat: add EmailCampaign model with status, format, and recipient types * feat: update SQLC version to v1.30.0 * feat: add SQL queries for email campaigns management * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: use sqlc.narg for nullable scheduled_at and sent_at in update queries Agent-Logs-Url: https://github.com/swamphacks/core/sessions/1e2e08e9-0737-47dd-8a01-bddead1ad570 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * style: group stdlib and third-party imports in email_campaigns repository Agent-Logs-Url: https://github.com/swamphacks/core/sessions/5c4475ce-1d02-429a-80bf-15b48fefcd07 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * feat: implement EmailCampaignService with CRUD operations and validation * Add email campaign HTTP routes and handler implementations * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * Applicant Views (Accepted, Rejected, Waitlisted) (#354) * started work on applicant views * added accepted, rejectd, waitlisted views * fix pnpm install fail inside docker * fix for pnpm install * maybe this will work * surely this time it'll work * 4th time is the charm * aaaaa * this should be it * confirm attendance, withdraw applications and small changes to settings page * feat/resume updating for applicants (#356) * feat: add functionality to replace uploaded resume in application form - Implemented a new component `ReplaceResume` to allow users to upload a new PDF resume. - Added a file input for selecting the resume and a button to trigger the upload. - Integrated `useReplaceResume` hook to handle the resume replacement API call. - Displayed success and error messages based on the upload status. * Fix PR review feedback for resume replacement flow --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * feat: register email campaign CRUD API and fix enum array handling (#357) * feat: add the existing email routes and handler to the API * feat: add email campaigns migration with status, format, and recipient types * feat: add EmailCampaign model with status, format, and recipient types * feat: update SQLC version to v1.30.0 * feat: add SQL queries for email campaigns management * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: use sqlc.narg for nullable scheduled_at and sent_at in update queries Agent-Logs-Url: https://github.com/swamphacks/core/sessions/1e2e08e9-0737-47dd-8a01-bddead1ad570 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * style: group stdlib and third-party imports in email_campaigns repository Agent-Logs-Url: https://github.com/swamphacks/core/sessions/5c4475ce-1d02-429a-80bf-15b48fefcd07 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * feat: implement EmailCampaignService with CRUD operations and validation * Add email campaign HTTP routes and handler implementations * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * feat: register/manually tested email campaign CRUD API and fix pgx enum array handling --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * Adds unit tests for the email campaign service’s validation and lifecycle rules. (#358) * feat: add the existing email routes and handler to the API * feat: add email campaigns migration with status, format, and recipient types * feat: add EmailCampaign model with status, format, and recipient types * feat: update SQLC version to v1.30.0 * feat: add SQL queries for email campaigns management * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: use sqlc.narg for nullable scheduled_at and sent_at in update queries Agent-Logs-Url: https://github.com/swamphacks/core/sessions/1e2e08e9-0737-47dd-8a01-bddead1ad570 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * style: group stdlib and third-party imports in email_campaigns repository Agent-Logs-Url: https://github.com/swamphacks/core/sessions/5c4475ce-1d02-429a-80bf-15b48fefcd07 Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * feat: implement EmailCampaignService with CRUD operations and validation * Add email campaign HTTP routes and handler implementations * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * feat: register/manually tested email campaign CRUD API and fix pgx enum array handling * added unit testing for email campaign api endpoints --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com> * Hieu/applications review (#359) * started working on application reviews * auto decision requests table improvements and other minor changes * improvements to the decision requests table and some changes idk * change review scales and added notes * application search and application responses viewer * application search and minor changes * Small refactor for application search page * applications backend refactor * application frontend refactor and some small tweaks to backend * add db generator * table component touchups * update gitignore * fix rebase error * more tweaks * remove openapi.json from git * remove schema.d.ts from git * added mlh badge and link to code of conduct --------- Co-authored-by: William Chi <williamc28275311@yahoo.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: williamchiii <188648862+williamchiii@users.noreply.github.com>
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds unit tests for the email campaign service’s validation and lifecycle rules.
The tests cover required campaign fields, whitespace-only input, editable campaign statuses, and timestamp requirements for scheduled and sent campaigns.
Type of Change
Checklist
Additional Notes
All email campaign service unit tests pass with:
go test ./internal/domains/email -v