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

Sort log update events by taken at time rather than event created at time #8046

Closed
khavinshankar opened this issue Jun 19, 2024 · 1 comment · Fixed by #8270
Closed

Sort log update events by taken at time rather than event created at time #8046

khavinshankar opened this issue Jun 19, 2024 · 1 comment · Fixed by #8270
Assignees

Comments

@khavinshankar
Copy link
Member

khavinshankar commented Jun 19, 2024

Describe the issue

When users are entering historical log updates into the system, they are often confused by the event log. The confusion arises because events are currently sorted by the date and time they were triggered, rather than by the "measured at" timestamp entered by the users during the log update. This makes it difficult for users to distinguish between past data and the latest changes.

To Replicate

  1. File a log update with the measured at date as today
  2. File a log update with the measured at date as yesterday or any past date
  3. See the events, it's currently sorted based on event creation date, which might be unintuitive for a normal user.

Screenshots

image

Suggested Solution

  • sort the events log date based on the "measured at" value
  • the field that doesn't have "measured at " value, sort them based on "created at" value

cc: @nihal467 @sainak @gigincg @aparnacoronasafe

Issue reported by athul doctor

Copy link

Hi, @coronasafe/care-frontend-maintainers, This issue has been automatically marked as stale because it has not had any recent activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants