Update shared_dashboards.md #26688
Closed
Update shared_dashboards.md #26688
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
neko-dd
reviewed
Dec 10, 2024
estherk15
added
the
under_review
Co-authored-by: Sandra (neko) <[email protected]>
|
@danielarodga Thanks for looking into this and updating the docs! Confirmed with the dashboards team, this is good to go! |
estherk15
removed
the
under_review
|
@danielarodga, could you reopen this PR with the following branch name: |
danielarodga
deleted the
danielarodriguez-update_shared_dashboards_refresh_rate
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do? What is the motivation?
Modifies the public refresh rate data shared with customers, as per this day it says that it takes around 30 seconds.
When taking a look at our internal documentation, it still says 30 seconds: https://datadoghq.atlassian.net/wiki/x/xwTGF (this is from May, 2020).
However, it seems like the code have been updated and now it has a note that it can take around 60 seconds:
https://github.com/DataDog/web-ui/blob/8258337c62d0705cacd19755c6a8196c5aba5d42/javascript/datadog/dashboard/components/Widget/WidgetAutoUpdateManager.tsx#L10
This caused confusion with my customer https://datadog.zendesk.com/agent/tickets/1955462, as it seems like they were expecting their public dashboard to refresh every 30 seconds, but when Support Agent was taking the time, it actually takes around 60 secs to be refreshed.
Merge instructions
Merge readiness:
Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the
<yourname>/descriptionnaming convention) and then add the following PR comment:Additional notes