Skip to content

Include FSA directory path in confirmation email to PI and requester#745

Merged
matthew-li merged 1 commit intodevelopfrom
fsa-confirmation-email-include-dir
Apr 10, 2026
Merged

Include FSA directory path in confirmation email to PI and requester#745
matthew-li merged 1 commit intodevelopfrom
fsa-confirmation-email-include-dir

Conversation

@matthew-li
Copy link
Copy Markdown
Collaborator

Description

**** Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. ****

  • Updated the confirmation email sent to the PI and requester when an FSA request is completed to include the path to the directory.

Type of change

**** Please delete options that are not relevant. ****

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

**** Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. ****

  • Test that the email includes the directory path (e.g., via Mailhog on port 8025).

PR Self Evaluation

Strikethrough things that don’t make sense for your PR.

  • My code follows the agreed upon best practices
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if needed)
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in the appropriate modules
  • I have performed a self-review of my own code

@matthew-li matthew-li merged commit fe2b3f5 into develop Apr 10, 2026
1 check passed
@matthew-li matthew-li deleted the fsa-confirmation-email-include-dir branch April 10, 2026 22:11
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