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

update SLO checklist page #22690

Merged
merged 4 commits into from
May 6, 2024
Merged

update SLO checklist page #22690

merged 4 commits into from
May 6, 2024

Conversation

roxanne-moslehi
Copy link
Contributor

@roxanne-moslehi roxanne-moslehi commented Apr 16, 2024

What does this PR do? What is the motivation?

Merge instructions

  • Please merge after reviewing

Additional notes

@roxanne-moslehi roxanne-moslehi requested a review from a team as a code owner April 16, 2024 21:45
@roxanne-moslehi
Copy link
Contributor Author

@estherk15 Two questions for you:

  1. Where does the info in the "Further Reading" section of pages get populated from? looking to update it
  2. This page links to a PDF - how do we update the PDF?

Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@cswatt cswatt added the editorial review Waiting on a more in-depth review label Apr 16, 2024
@cswatt
Copy link
Contributor

cswatt commented Apr 16, 2024

Added DOCS-7738 to track

@estherk15
Copy link
Contributor

@roxanne-moslehi

  1. You can add links at the top of the markdown file here
  2. I think that PDF was populated by the previous SLO PM.

@estherk15 estherk15 merged commit d572933 into master May 6, 2024
11 of 12 checks passed
@estherk15 estherk15 deleted the rox/slo-checklist branch May 6, 2024 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial review Waiting on a more in-depth review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants