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

Add investigate and timeline page #25889

Merged
merged 4 commits into from
Oct 25, 2024

Conversation

estherk15
Copy link
Contributor

@estherk15 estherk15 commented Oct 23, 2024

What does this PR do? What is the motivation?

This reorganize the Incident details page

  • Rename to Investigate Incidents
  • Break out the Timeline into a separate page
  • Remove the Remediation, Response Team, and Notification sections, these were or will be transferred to other pages.
  • Add an alias and update reference links
  • DOCS-8293

Merge instructions

  • Please merge after reviewing

Additional notes

@estherk15 estherk15 requested a review from a team as a code owner October 23, 2024 20:19
@github-actions github-actions bot added Architecture Everything related to the Doc backend Images Images are added/removed with this PR labels Oct 23, 2024
@estherk15 estherk15 added the editorial review Waiting on a more in-depth review label Oct 23, 2024

| Content type | Description |
| ------------------ | ------------------------------------------------------------------------------------------------------ |
| Responder note | A note manually written by an incident responder. Responder notes have the following sub-types:<br>- *Graph*: The responder note contains one or more Datadog graphs<br>- *Link*: The responder note contains a hyperlink<br>- *Code*: The responder note contains text wrapped in Markdown syntax for code blocks
Copy link
Contributor

Choose a reason for hiding this comment

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

Does a responder note always have to be of one of the subtypes, or is there just a "plain" responder note type?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You can have a plain note, within a note cell you can add graphs and links. If you don't add anything, it'd just be a responder note.

@estherk15 estherk15 requested a review from neko-dd October 25, 2024 12:42
@estherk15
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Oct 25, 2024

🚂 MergeQueue: pull request added to the queue

The median merge time in master is 25m.

Use /merge -c to cancel this operation!

@dd-mergequeue dd-mergequeue bot merged commit 98faa2a into master Oct 25, 2024
20 of 24 checks passed
@dd-mergequeue dd-mergequeue bot deleted the esther/docs-8293-investigate-incident branch October 25, 2024 18:43
theraffoul pushed a commit that referenced this pull request Nov 25, 2024
* Add investigate and timeline page

* Apply suggestions from code review

Co-authored-by: Sandra (neko) <[email protected]>

* Add Timeline docs referenc

* Add reference link number

---------

Co-authored-by: Sandra (neko) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Architecture Everything related to the Doc backend editorial review Waiting on a more in-depth review Images Images are added/removed with this PR mergequeue-status: done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants