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

Move Timestamp into SendStatus Component #2747

Closed
tdurnford opened this issue Dec 17, 2019 · 0 comments · Fixed by #2786
Closed

Move Timestamp into SendStatus Component #2747

tdurnford opened this issue Dec 17, 2019 · 0 comments · Fixed by #2786
Assignees

Comments

@tdurnford
Copy link
Contributor

tdurnford commented Dec 17, 2019

Feature Request

Moving the Timestamp component into the SendStatus component would reduce duplicate code in the StackedLayout andCarousalLayout components. We should also remove the the Timestamp style set because it is almost identical to the Timestamp style set.

Describe alternatives you have considered
No.

Additional context
Simplifies #2722

[Enhancement]

@tdurnford tdurnford added Enhancement customer-reported Required for internal Azure reporting. Do not delete. Bot Services Required for internal Azure reporting. Do not delete. Do not change color. labels Dec 17, 2019
@tdurnford tdurnford self-assigned this Dec 17, 2019
@tdurnford tdurnford removed Bot Services Required for internal Azure reporting. Do not delete. Do not change color. customer-reported Required for internal Azure reporting. Do not delete. labels Dec 17, 2019
@corinagum corinagum added R8 and removed Pending labels Dec 17, 2019
@tdurnford tdurnford changed the title Move SendStatus into Timestamp Component Move Timestamp into SendStatus Component Dec 31, 2019
@compulim compulim mentioned this issue Mar 5, 2020
40 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants