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: [DI-23411] - Updated preference key for date time range picker #11631

Merged
merged 3 commits into from
Feb 11, 2025

Conversation

nikhagra-akamai
Copy link
Contributor

@nikhagra-akamai nikhagra-akamai commented Feb 7, 2025

Description 📝

Changed preference key for date time range picker because preference saved with previous time range select has different structure to save preference and it may impact user experience who are first time using date time range picker with preferences have data from older time range select.

Changes 🔄

List any change(s) relevant to the reviewer.

  1. Updated constants to change values of TIME_DURATION

Target release date 🗓️

10 Feb

Preview 📷

Error because of bug
Screenshot 2025-02-06 at 10 55 23 PM

How to test 🧪

Prerequisites

(How to setup test environment)

  • ...
  • ...

Reproduction steps

(How to reproduce the issue, if applicable)

  1. Go to cloud.linode.com/monitor/dashboards.
  2. Change value in time range dropdown and see the request data in network tab for preferences api
  3. You'll see its value for timeDuration key is a string value based on previous time range select but with new date time range picker it should be an object of {end: string, start: string, preset: string}.

Verification steps

(How to verify changes)

  • ...
  • ...
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@nikhagra-akamai nikhagra-akamai requested a review from a team as a code owner February 7, 2025 13:57
@nikhagra-akamai nikhagra-akamai requested review from bnussman-akamai and cliu-akamai and removed request for a team February 7, 2025 13:57
Copy link

github-actions bot commented Feb 7, 2025

Coverage Report:
Base Coverage: 78.94%
Current Coverage: 78.94%

@nikhagra-akamai
Copy link
Contributor Author

@bnussman-akamai @cliu-akamai can we have second approval on this PR

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 501 passing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing501 Passing2 Skipped98m 37s

@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Cloud Pulse labels Feb 10, 2025
@nikhagra-akamai nikhagra-akamai merged commit ce120b2 into linode:develop Feb 11, 2025
22 of 23 checks passed
Copy link

cypress bot commented Feb 11, 2025

Cloud Manager E2E    Run #7206

Run Properties:  status check passed Passed #7206  •  git commit ce120b24d8: fix: [DI-23411] - Updated preference key for date time range picker (#11631)
Project Cloud Manager E2E
Branch Review develop
Run status status check passed Passed #7206
Run duration 02m 07s
Commit git commit ce120b24d8: fix: [DI-23411] - Updated preference key for date time range picker (#11631)
Committer Nikhil Agrawal
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 5
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 501
View all changes introduced in this branch ↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Multiple approvals and ready to merge!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants