Skip to content

Added issue warning, implemented emails on endpoints#110

Merged
Prakhar896 merged 11 commits intomainfrom
junhan
Aug 7, 2024
Merged

Added issue warning, implemented emails on endpoints#110
Prakhar896 merged 11 commits intomainfrom
junhan

Conversation

@JunHammy
Copy link
Copy Markdown
Contributor

@JunHammy JunHammy commented Aug 2, 2024

What was done in this PR

  1. Fixed small bugs
  2. Added /issueWarning and unflagHost endpoints
  3. Implemented emails on CreateAccount, VerifyEmail, AccountRecovery and IssueWarning

Sensitive changes

  1. Added flaggedForHygiene to Host model

@JunHammy JunHammy requested a review from Prakhar896 August 2, 2024 21:57
@JunHammy JunHammy added the enhancement New feature or request label Aug 2, 2024
Comment thread models/Host.js Outdated
Comment thread routes/identity/AccountRecovery.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/CreateAccount.js
Comment thread routes/identity/emailVerification.js Outdated
@JunHammy JunHammy requested a review from Prakhar896 August 3, 2024 17:19
Copy link
Copy Markdown
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really well-done. Minor changes needed.

Comment thread routes/cdn/coreData.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread views/emails/hygieneWarningReceived.html Outdated
@JunHammy JunHammy requested a review from Prakhar896 August 7, 2024 01:24
Copy link
Copy Markdown
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor changes needed

Comment thread routes/identity/Admin/HygieneReports.js
Comment thread routes/identity/Admin/HygieneReports.js Outdated
Comment thread routes/identity/Admin/HygieneReports.js Outdated
@JunHammy JunHammy requested a review from Prakhar896 August 7, 2024 03:09
Copy link
Copy Markdown
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Prakhar896 Prakhar896 merged commit 72fef11 into main Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants