Skip to content

Conversation

joanagmaia
Copy link
Contributor

@joanagmaia joanagmaia commented Aug 28, 2023

Changes proposed ✍️

Disclaimer added to both Dashboard and Reports next to the pages title:

  • "Data on this page is refreshed every 15 min."

What

🤖 Generated by Copilot at 31b732c

The pull request modifies two components, dashboard-filters.vue and report-template-filters.vue, to add a text message and more filtering options for the dashboard data. The text message informs the user that the dashboard data is not real-time, while the filtering options allow the user to select different criteria for the data. The changes aim to improve the user experience, clarity, consistency, and maintainability of the dashboard and report features.

🤖 Generated by Copilot at 31b732c

dashboard-filters
More options and clarity
Autumn leaves falling

Why

How

🤖 Generated by Copilot at 31b732c

  • Added a text message element to the flex container of the dashboard and report filters components to inform the user that the data is not real-time (link, link)
  • Refactored the dashboard-filters.vue component to reuse the logic and layout of the report-template-filters.vue component, which allows the user to filter the data by platform, team members, and team activities (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@joanagmaia joanagmaia added the Improvement Created by Linear-GitHub Sync label Aug 28, 2023
@joanagmaia joanagmaia self-assigned this Aug 28, 2023
<div class="flex items-center gap-9">
<div class=" text-sm flex items-center gap-2">
<i class="text-gray-500 ri-time-line text-base" />
<span class="text-gray-500">Data on this page refreshed every hour</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

10 minutes 🙂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated it to 15 as discussed. Will merge and deploy it

@joanagmaia joanagmaia requested a review from joanreyero August 29, 2023 14:10
@joanagmaia joanagmaia merged commit 165662c into main Aug 29, 2023
@joanagmaia joanagmaia deleted the improvement/dashboard-disclaimer branch August 29, 2023 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants