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

Show published date again #2675

Closed
xaur opened this issue Nov 24, 2021 · 4 comments · Fixed by #2680
Closed

Show published date again #2675

xaur opened this issue Nov 24, 2021 · 4 comments · Fixed by #2680
Assignees
Labels
bug A bug that made it into a production environment. pi-not-deployed This item has been merged into master, but has not been deployed to production yet.

Comments

@xaur
Copy link

xaur commented Nov 24, 2021

Published date is an important piece of info I often look at. It was shown on the old UI:

pi-pubdate-1

but missing from the new UI:

pi-pubdate-2

I can get this info from dcrdata:

pi-pubdate-3

But miss having it directly on Pi when surfing around and researching stuff.

@lukebp
Copy link
Member

lukebp commented Nov 24, 2021

This was suppose to be fixed back in June on #2430.

@tiagoalvesdulce @victorgcramos what PR broke this and why are we having to fix the same exact issue, multiple times within months of each other?

@lukebp lukebp added 91cfcc8 bug A bug that made it into a production environment. labels Nov 24, 2021
@lukebp
Copy link
Member

lukebp commented Nov 24, 2021

Introduced by #2454 . Let's make sure to add e2e tests for the fix this time.

@victorgcramos
Copy link
Member

victorgcramos commented Nov 29, 2021

As discussed on Matrix, we should include the timestamps also for mobile.

@victorgcramos
Copy link
Member

On it!

@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
bug A bug that made it into a production environment. pi-not-deployed This item has been merged into master, but has not been deployed to production yet.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants