Skip to content

[SS-2218] Email UI Fixes#401

Merged
utkarsh14 merged 2 commits intodevfrom
email_ui
Dec 24, 2025
Merged

[SS-2218] Email UI Fixes#401
utkarsh14 merged 2 commits intodevfrom
email_ui

Conversation

@utkarsh14
Copy link
Collaborator

@utkarsh14 utkarsh14 commented Nov 19, 2025

SS-2218 Email UI Fixes

Ticket

Fixes: https://idinsight.atlassian.net/browse/SS-2218

Description, Motivation and Context

Email UI fixes:

  • Standard UI changes for bordered table, custom button
  • Changed email time to be in 12 hour format
  • Fixed email delivery report UI, added filters on table
  • Fetch enumerator level report when user switches date in delivery report.

How Has This Been Tested?

Local

Checklist:

This checklist is a useful reminder of small things that can easily be forgotten.
Put an x in all the items that apply and remove any items that are not relevant to this PR.

  • My code follows the style guidelines of this project
  • I have reviewed my own code to ensure good quality
  • I have tested the functionality of my code to ensure it works as intended
  • I have resolved merge conflicts
  • I have updated the automated tests (if applicable)
  • I have written [good commit messages][1]
  • I have updated the README file (if applicable)
  • I have updated affected documentation (if applicable)

@utkarsh14 utkarsh14 changed the title Email UI Fixes [SS-2218] Email UI Fixes Nov 26, 2025
@utkarsh14 utkarsh14 requested a review from jeenut27 November 26, 2025 05:38
Copy link
Collaborator

@jeenut27 jeenut27 left a comment

Choose a reason for hiding this comment

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

This looks good. I was testing the email UI and noticed a couple of things:

  1. On the Email Delivery Report popup, if a schedule doesn't have any emails - the table continues to show the data from the previously selected schedule
  2. The Trigger Time on manual triggers screen still shows a full date format. We had discussed updating this to date and time for uniformity. The View Recipients button here can also be labelled view delivery report for uniformity.

Approving! Feel free to merge

@utkarsh14
Copy link
Collaborator Author

image

Fixed trigger dae times and view recipient button.

schedule doesn't have any emails - the table continues to show the data from the previously selected schedule

Both tables had same data and first fetch was not working i have added an use effect to fix it.

@utkarsh14 utkarsh14 merged commit 841b070 into dev Dec 24, 2025
3 checks passed
@utkarsh14 utkarsh14 deleted the email_ui branch December 24, 2025 05:52
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

Comments