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.

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

[MDS-5836] trigger notification report requested #3087

Merged
merged 3 commits into from
May 10, 2024

Conversation

isuru-aot
Copy link
Contributor

Objective

MDS-5836

Why are you making this change? Provide a short explanation and/or screenshots
Screenshot 2024-05-07 165735
Screenshot 2024-05-07 165749

henryoforeh-dev
henryoforeh-dev previously approved these changes May 8, 2024
permit_info_value = self.permit_number + ": "

subject = "A Report is requested in MineSpace"
due_date = due_date = (self.due_date).strftime("%b %d %Y") if self.due_date else "N/A"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
due_date = due_date = (self.due_date).strftime("%b %d %Y") if self.due_date else "N/A"
due_date = (self.due_date).strftime("%b %d %Y") if self.due_date else "N/A"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@matbusby-fw need your approval again for this.

Copy link
Collaborator

@matbusby-fw matbusby-fw left a comment

Choose a reason for hiding this comment

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

Awesome work! Just noticed one double assignment of a due date in there.

Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_core-api'

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@isuru-aot isuru-aot merged commit 224bc10 into develop May 10, 2024
11 of 12 checks passed
@isuru-aot isuru-aot deleted the mds-5836-trigger-notification-report-requested branch May 10, 2024 18:07
simensma-fresh pushed a commit that referenced this pull request Jun 12, 2024
* email notification for report requests

* fixing permit view

* fix typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants