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.

Sign up for GitHub

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

Jump to bottom

Clarify 5-min lookback window for Delivery Overview #6544

Merged
merged 3 commits into from
Nov 14, 2024
Merged

Clarify 5-min lookback window for Delivery Overview #6544

merged 3 commits into from
Nov 14, 2024

Conversation

joeynmq
Copy link
Contributor

@joeynmq joeynmq commented May 15, 2024

Proposed changes

To clarify in our public documentation that our Delivery Overview metrics has a 5-minute lookback window so that new events coming from sources will not be immediately populated.

Merge timing

ASAP once approved

Related issues (optional)

@joeynmq joeynmq added the KCS label May 15, 2024
@joeynmq joeynmq marked this pull request as ready for review August 27, 2024 05:42
@joeynmq joeynmq requested a review from a team as a code owner August 27, 2024 05:42
@joeynmq joeynmq requested a review from pwseg August 27, 2024 05:42
pwseg
pwseg previously approved these changes Nov 14, 2024
View reviewed changes
@seg-atlantis-prod
Copy link

Plan Error

running git merge -q --no-ff -m atlantis-merge FETCH_HEAD: Auto-merging src/connections/delivery-overview.md
CONFLICT (content): Merge conflict in src/connections/delivery-overview.md
Automatic merge failed; fix conflicts and then commit the result.
: exit status 1

@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Nov 14, 2024
@pwseg pwseg merged commit bd8ec71 into develop Nov 14, 2024
5 checks passed
@pwseg pwseg deleted the joeynmq-patch-5 branch November 14, 2024 04:22
@github-actions GitHub Actions
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pwseg pwseg pwseg approved these changes

Assignees
No one assigned
Labels
content-update updates to content that are not new features, includes grammar fixes, added notes KCS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joeynmq @pwseg