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

Improvement in renderDay #2227

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Hassan-Riaz
Copy link

@Hassan-Riaz Hassan-Riaz commented Jun 29, 2022

Placing date hiding logic to generic place so the person who is integrating doesn't have to apply same logic again.

Render Day functionality should be generic as when we renderDay handler is provided from front end then this logic is duplicated
@Hassan-Riaz Hassan-Riaz changed the title Update Message.tsx Improvement in renderDay Jul 2, 2022
Copy link
Collaborator

@Johan-dutoit Johan-dutoit left a comment

Choose a reason for hiding this comment

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

Seems sensible. Can you share an example of the before and after?

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