Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(EMS-1269): no PDF - CRON - Submission deadline email #2670

Merged
merged 2 commits into from
Jul 2, 2024
Merged

fix(EMS-1269): no PDF - CRON - Submission deadline email #2670

merged 2 commits into from
Jul 2, 2024

Conversation

Zainzzkk
Copy link
Collaborator

@Zainzzkk Zainzzkk commented Jul 2, 2024 •
edited
Loading

Introduction ✏️

This PR fixes an issue with the submission deadline email where the number of emails sent could not be compared leading to an error

Resolution ✔️

  • added await and constant to Promise.all
  • returned constant
  • Updated tests

@Zainzzkk Zainzzkk added the fix Fixes an existing issue label Jul 2, 2024
@Zainzzkk Zainzzkk self-assigned this Jul 2, 2024
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@Zainzzkk Zainzzkk requested a deployment to qa July 2, 2024 17:00 — with GitHub Actions Abandoned
@ttbarnes ttbarnes merged commit 743388c into main-application-no-pdf Jul 2, 2024
166 of 168 checks passed
@ttbarnes ttbarnes deleted the fix/EMS-1269 branch July 2, 2024 17:10
@abhi-markan abhi-markan mentioned this pull request Jul 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ttbarnes ttbarnes ttbarnes approved these changes

Assignees

@Zainzzkk Zainzzkk

Labels
fix Fixes an existing issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Zainzzkk @ttbarnes