Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add display mappings for delivery status and channel type #117

Merged
merged 2 commits into from
Jan 16, 2024

Conversation

ghost
Copy link

@ghost ghost commented Jan 12, 2024

Description:
This PR adds display string mappings for showing easily readable strings for Channel Type and Delivery Status in the portal instead of numeric values.

Related changes:

  • Add deliveryStatusMap and channelTypeMap for mapping display strings for corresponding values
  • Update notifications.component html to use these maps when displaying delivery status and channel type values, along with proper styles

Screenshots:
image

Add deliveryStatusMap and channelTypeMap for mapping display strings for corresponding values
Update notifications.component html to use these maps when displaying delivery status and channel type values
@ghost ghost added the enhancement New feature or request label Jan 12, 2024
@ghost ghost self-assigned this Jan 12, 2024
Copy link
Collaborator

@xixas xixas left a comment

Choose a reason for hiding this comment

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

@LakshayaT
Add teplate formatting to status for example status field here.
https://primeng.org/table#template
https://www.primefaces.org/primeng-v14-lts/table/templating

Update notifications.scss to add styles for displaying statuses as badges
Update notifcations.component ts file to add style string to the deliveryStatusMap
Update notifications.component html file to properly display status strings using style and value strings
@ghost
Copy link
Author

ghost commented Jan 16, 2024

@LakshayaT Add teplate formatting to status for example status field here. https://primeng.org/table#template https://www.primefaces.org/primeng-v14-lts/table/templating

This has been implemented in c12d32a.

@ghost ghost added the ready for review label Jan 16, 2024
@xixas xixas merged commit b021c28 into main Jan 16, 2024
8 checks passed
@xixas xixas deleted the feat/display-mapping branch January 16, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant