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

Add an Updated timestamp to MyFacilities pages #1259

Merged
merged 7 commits into from Aug 28, 2020

Conversation

kitallis
Copy link
Contributor

@kitallis kitallis commented Aug 28, 2020

Story card: https://app.clubhouse.io/simpledotorg/story/1103/add-an-updated-timestamp-to-myfacilities-pages

Because

We're changing how we refresh matviews weekly to figure out perf bottlnecks. In the interim, we'll display an page update timestamp to let people know if it might be stale.

This addresses

Desktop view

Screenshot 2020-08-28 at 12 42 14 PM
Screenshot 2020-08-28 at 12 42 06 PM

Mobile view

Screenshot 2020-08-28 at 12 42 02 PM

Super edge-case when there's no timestamp present
Screenshot 2020-08-28 at 12 41 47 PM

@harimohanraj89 harimohanraj89 temporarily deployed to simple-review-pr-1259 August 28, 2020 07:15 Inactive
@kitallis kitallis temporarily deployed to simple-review-pr-1259 August 28, 2020 07:29 Inactive
@harimohanraj89 harimohanraj89 temporarily deployed to simple-review-pr-1259 August 28, 2020 07:48 Inactive
@harimohanraj89 harimohanraj89 temporarily deployed to simple-review-pr-1259 August 28, 2020 07:50 Inactive
@harimohanraj89 harimohanraj89 temporarily deployed to simple-review-pr-1259 August 28, 2020 10:54 Inactive
@harimohanraj89 harimohanraj89 temporarily deployed to simple-review-pr-1259 August 28, 2020 10:56 Inactive
@harimohanraj89 harimohanraj89 temporarily deployed to simple-review-pr-1259 August 28, 2020 11:00 Inactive
Copy link
Contributor

@harimohanraj89 harimohanraj89 left a comment

Choose a reason for hiding this comment

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

:shipit:

@harimohanraj89 harimohanraj89 merged commit c63383a into master Aug 28, 2020
@harimohanraj89 harimohanraj89 deleted the timestamp-my-facilities branch August 28, 2020 11:26
@rsanheim rsanheim mentioned this pull request Aug 28, 2020
13 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
Development

Successfully merging this pull request may close these issues.

None yet

3 participants