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

i1023/i1005 - set up maildev for staging #2199

Merged
merged 7 commits into from
May 1, 2024

Conversation

ShanaLMoore
Copy link
Collaborator

@ShanaLMoore ShanaLMoore commented May 1, 2024

This commit sets up maildev for staging and demo. It also temporarily enables email notifications, for testing purposes. This change will be removed after confirmation.

Issues:

Created this work that requires an approval.

Notification email received in mail dev:

Screenshot 2024-05-01 at 11-46-42 MailDev

This commit sets up maildev for staging. It also temporarily enables email for notifications, for testing purposes. This change will be removed after confirmation.

Issue:
- scientist-softserv/palni-palci#1023
@ShanaLMoore ShanaLMoore force-pushed the i1005-setup-email-notifications branch from e09f9ca to 1d8b975 Compare May 1, 2024 15:35
@ShanaLMoore ShanaLMoore added the patch-ver for release notes label May 1, 2024
@ShanaLMoore ShanaLMoore force-pushed the i1005-setup-email-notifications branch from e50964a to f1fe2b6 Compare May 1, 2024 18:08
@ShanaLMoore ShanaLMoore changed the title i1023 - set up maildev for staging i1023/i1005 - set up maildev for staging May 1, 2024
@ShanaLMoore ShanaLMoore marked this pull request as ready for review May 1, 2024 19:04
Copy link
Collaborator

@kirkkwang kirkkwang left a comment

Choose a reason for hiding this comment

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

Cool looks good!

@ShanaLMoore ShanaLMoore merged commit 83d105f into main May 1, 2024
4 checks passed
@ShanaLMoore ShanaLMoore deleted the i1005-setup-email-notifications branch May 1, 2024 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants