bito-code-review[bot] commented on PR #16452:
URL: https://github.com/apache/grails-core/pull/16452#issuecomment-5928312352

   <!-- Bito Reply -->
   The changes introduce more granular control over email overrides by 
splitting the single `overrideAddress` property into `overrideToAddress` and 
`overrideFromAddress`. This is a breaking change for users who relied on the 
single `overrideAddress` to control both fields, as the logic now prioritizes 
the specific override properties if they are set. An upgrade note is 
recommended to inform users that they should migrate to the new configuration 
properties if they require specific control over 'to' or 'from' addresses.
   
   **grails-mail/src/main/groovy/grails/plugins/mail/MailMessageBuilder.groovy**
   ```
   this.overrideToAddress = properties.overrideToAddress ?: overrideAddress
           this.overrideFromAddress = properties.overrideFromAddress ?: 
overrideAddress
           this.defaultFrom = overrideFromAddress ?: properties.default.from
           this.defaultTo = overrideToAddress ?: properties.default.to
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to