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

Plugin - Alert Messenger - Implement (I4I submission) #694

Open
wants to merge 22 commits into
base: dev
Choose a base branch
from
Open

Plugin - Alert Messenger - Implement (I4I submission) #694

wants to merge 22 commits into from

Conversation

GregJohnStewart
Copy link
Contributor

Checklist:

  • Tested

@GregJohnStewart GregJohnStewart changed the title Plugin - Alert Messenger - Now on Quarkus 3.13.3 Plugin - Alert Messenger - Implement (I4I submission) Sep 18, 2024
@BrendanAndrews
First commit
04f1c0a
@BrendanAndrews
First successful test of mock email sending
7434721
@BrendanAndrews
Fixing email address
1787108
@BrendanAndrews
Adding SMTP Server configuration and real email testing (even though …
e20f850 
…it's commented out).
@BrendanAndrews
Adding some additional comments for easier reading
b6211a9
@BrendanAndrews
Pushing Slack function and test code (commented out test code)
4d5c4ab
@BrendanAndrews
Fixing some issue with getting information from the token
7ea4180
@BrendanAndrews
Database connection works
ac01608
@BrendanAndrews
Fixing my credentials that I accidentally pushed with new changed ones
ac38968
@BrendanAndrews
Fixing environment variables
ca3fc17
@BrendanAndrews
Storing slack webhook url and whether the user wants to receive email…
1db77b2 
… notifications within our database. A form inserts those preferences. All is working.
@BrendanAndrews
Created AlertConsumer.java business logic for sending messages.
860a208
@BrendanAndrews
I got the AlertConsumer.java file working in a mock test.
b8b69fe
@BrendanAndrews
Version of the Alert Messenger that's fully functional
71ca97c
@BrendanAndrews
Removing unnecessary commented out code
74f47a9
@BrendanAndrews
Removing ExampleConsumer and TestConsumerTest
cb5d101
@BrendanAndrews
Added TEST_USER_ID environment variable for automated Slack message t…
0253c4c 
…esting
@BrendanAndrews
Adding SMTP_USERNAME environment variable to allow other SMTP servers…
9d973cb 
… to be plugged in in the future
@BrendanAndrews
Added comments to my code
d82fcb2
@GregJohnStewart GregJohnStewart marked this pull request as ready for review December 8, 2024 19:47
@GregJohnStewart
Copy link
Contributor Author

@all-contributors
please add @BrendanAndrews for code
please add @DanielKrejska for code
please add @LouisBarbier for content and project management

@allcontributors allcontributors bot mentioned this pull request Dec 8, 2024
Merged
@allcontributors AllContributors
Copy link
Contributor

@GregJohnStewart

I've put up a pull request to add @BrendanAndrews! 🎉

@GregJohnStewart
Copy link
Contributor Author

@all-contributors
please add @DanielKrejska for code

@allcontributors allcontributors bot mentioned this pull request Dec 8, 2024
Merged
@allcontributors AllContributors
Copy link
Contributor

@GregJohnStewart

I've put up a pull request to add @DanielKrejska! 🎉

@GregJohnStewart
Copy link
Contributor Author

@all-contributors
please add @LouisBarbier for content and project management

@allcontributors allcontributors bot mentioned this pull request Dec 8, 2024
Merged
@allcontributors AllContributors
Copy link
Contributor

@GregJohnStewart

I've put up a pull request to add @LouisBarbier! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GregJohnStewart @BrendanAndrews