Skip to content

[PB-1960]: feat/implement mail address decorator and update email controller#30

Open
jzunigax2 wants to merge 1 commit intomasterfrom
fix/mail-address-resolution
Open

[PB-1960]: feat/implement mail address decorator and update email controller#30
jzunigax2 wants to merge 1 commit intomasterfrom
fix/mail-address-resolution

Conversation

@jzunigax2
Copy link
Copy Markdown
Contributor

Resolve requesting user's mail address to their default email address. Would need a future task to tackle the scenario where one user could have many email addrresses

  • Added mailAddressFactory as a parameter decorator to extract MailAddress from the request context.
  • Created unit tests for the mail address decorator to ensure correct functionality.
  • Updated EmailController to use the new MailAddress decorator for email parameter extraction, replacing the previous user email extraction method.
  • Enhanced MailAccountGuard to attach the default mail address to the request context and handle cases where no default address is available.

- Added `mailAddressFactory` as a parameter decorator to extract MailAddress from the request context.
- Created unit tests for the mail address decorator to ensure correct functionality.
- Updated `EmailController` to use the new `MailAddress` decorator for email parameter extraction, replacing the previous user email extraction method.
- Enhanced `MailAccountGuard` to attach the default mail address to the request context and handle cases where no default address is available.
@jzunigax2 jzunigax2 self-assigned this Apr 24, 2026
@jzunigax2 jzunigax2 added the enhancement New feature or request label Apr 24, 2026
@jzunigax2 jzunigax2 changed the title []: feat/implement mail address decorator and update email controller [PB-1960]: feat/implement mail address decorator and update email controller Apr 24, 2026
@sonarqubecloud
Copy link
Copy Markdown

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.

1 participant