Skip to content

Conversation

nalduu
Copy link
Contributor

@nalduu nalduu commented May 22, 2025

migrated the submission acknowledgement code to a new file, added the acknowledgement email template

@nalduu nalduu force-pushed the publish_ack branch 2 times, most recently from fca1a1f to 90ed131 Compare June 4, 2025 14:35
Copy link
Contributor

@kngai kngai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some feedback for duplicates and one variable to rename

@kngai kngai force-pushed the publish_ack branch 3 times, most recently from 89df941 to 0e49ddd Compare June 11, 2025 19:37
… acknowledgement email template

- Update default.env
- Update requirements.txt
- Fix up URLs in template files
- Rename WDR_TEMPLATE_PATH to WDR_FEEDBACK_TEMPLATE_PATH
- More verbose debugging and code cleanup
@kngai kngai merged commit 4c9d625 into woudc:master Jun 11, 2025
1 check passed
@coveralls
Copy link

Coverage Status

coverage: 49.203% (+0.05%) from 49.156%
when pulling 0e08558 on nalduu:publish_ack
into f1c9124 on woudc:master.

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.

3 participants