Skip to content

IVS-589 - Add channel to admin ack emails (LOW PRIO)#208

Merged
rw-bsi merged 1 commit intodevelopmentfrom
IVS-589_Add_Channel_to_Admin_Ack_Emails
Aug 4, 2025
Merged

IVS-589 - Add channel to admin ack emails (LOW PRIO)#208
rw-bsi merged 1 commit intodevelopmentfrom
IVS-589_Add_Channel_to_Admin_Ack_Emails

Conversation

@rw-bsi
Copy link
Contributor

@rw-bsi rw-bsi commented Jul 25, 2025

No description provided.

@rw-bsi rw-bsi requested a review from civilx64 July 25, 2025 14:54
@rw-bsi rw-bsi self-assigned this Jul 25, 2025
Copy link
Contributor

@civilx64 civilx64 left a comment

Choose a reason for hiding this comment

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

@rw-bsi Curious, why include the channel in only 1 of the 3 message locations? Is there a reason to not include it in each location?

@rw-bsi
Copy link
Contributor Author

rw-bsi commented Jul 30, 2025

@rw-bsi Curious, why include the channel in only 1 of the 3 message locations? Is there a reason to not include it in each location?

Only in internal administrative emails (called 'ack') is it useful to know imo. End users know how they submitted it.
(one file was changed just to reflect 'uploaded' vs 'submitted' as new wording since API came in the picture).

@civilx64
Copy link
Contributor

That make sense, thanks!

@rw-bsi rw-bsi merged commit 1227efc into development Aug 4, 2025
3 checks passed
@rw-bsi rw-bsi deleted the IVS-589_Add_Channel_to_Admin_Ack_Emails branch August 4, 2025 13:03
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.

2 participants