Purpose

The purpose of this PR is to fix #388

The purpose of this pull request is to address the ExcessiveMethodLength and DuplicateCode problems in the sendMenteeApplicationEmails method to improve readability and make the code easier to maintain.

Goals

• Reduce the complexity of the sendMenteeApplicationEmails method.
• Eliminate code duplication within the method.
• Improve the testability of the code by separating the logic into smaller methods.

Approach

Application of the "Extract Method" refactoring technique:

• Creation of buildApprovedMenteeEmailMessage and buildRejectedMenteeEmailMessage methods to build email messages.
• Extraction of the email sending and saving logic for the sendAndSaveMenteeApplicationEmail method.

Replacement of duplicate operations in the sendMenteeApplicationEmails method with calls to the new methods.

Checklist

• This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
• I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation