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

fix: show proposal published and edited dates #2680

Merged

Conversation

victorgcramos
Copy link
Member

Closes #2675

This commit displays the missing "published" timestamp event for
proposals that have been edited. Also, display timestamps for mobile
views and adds e2e tests to prevent this from happening further.

UI Changes Screenshot

Regular view:

List:
Screen Shot 2021-11-29 at 9 03 10 PM

Details:
Screen Shot 2021-11-29 at 9 04 36 PM

Mobile View

List:
Screen Shot 2021-11-29 at 9 02 55 PM

Details:
Screen Shot 2021-11-29 at 9 03 21 PM

This commit displays the missing "published" timestamp event for
proposals that have been edited. Also, display timestamps for mobile
views and adds e2e tests to prevent this from happening further.
@victorgcramos victorgcramos changed the title fix: show proposal published and edit dates fix: show proposal published and edited dates Nov 30, 2021
Copy link
Member

@tiagoalvesdulce tiagoalvesdulce left a comment

Choose a reason for hiding this comment

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

tACK

Copy link
Member

@lukebp lukebp left a comment

Choose a reason for hiding this comment

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

ACK

@tiagoalvesdulce tiagoalvesdulce merged commit 8f4a9c1 into decred:master Dec 19, 2021
@tiagoalvesdulce tiagoalvesdulce added the pi-not-deployed This item has been merged into master, but has not been deployed to production yet. label Dec 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pi-not-deployed This item has been merged into master, but has not been deployed to production yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show published date again
3 participants