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

MAIN-T-99 Send registration email async #98

Closed
wants to merge 2 commits into from

Conversation

hanna-eismant
Copy link
Owner

Add event-driven asynchronous email notifications

Added an asynchronous event-driven mechanism for sending email notifications after user registration. This was accomplished by publishing a UserRegistrationEvent directly after the user is created. The event is then consumed by a listener, DokyEventListener, which triggers the sending of the email. To accommodate this, the Awaitility library has also been added to the project to allow for testing of asynchronous processes. The build.gradle file was updated accordingly.

Add event-driven asynchronous email notifications

Added an asynchronous event-driven mechanism for sending email notifications after user registration. This was accomplished by publishing a `UserRegistrationEvent` directly after the user is created. The event is then consumed by a listener, `DokyEventListener`, which triggers the sending of the email. To accommodate this, the Awaitility library has also been added to the project to allow for testing of asynchronous processes. The `build.gradle` file was updated accordingly.
@hanna-eismant hanna-eismant self-assigned this Apr 26, 2024
Copy link

github-actions bot commented Apr 26, 2024

Qodana for JVM

10 new problems were found

Inspection name Severity Problems
Incorrect autowiring in Spring bean components 🔴 Failure 1
Vulnerable API usage 🔶 Warning 4
Unused symbol 🔶 Warning 3
String template as argument to logging call 🔶 Warning 1
Accessor call that can be replaced with property access syntax ◽️ Notice 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

The `LOG` instance, which was used for logging purposes, is no longer needed in the `JwtProvider` class. Consequently, the corresponding declaration has been deleted to clean up the code and reduce clutter.
@hanna-eismant hanna-eismant deleted the MAIN-T-99-Send-registration-email-async branch April 26, 2024 23:02
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.

1 participant