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

docs: [AH-809]: Add AH troubleshooting section and horizontal sidebars #8950

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

docs: [AH-809]: Add AH troubleshooting section and horizontal sidebars #8950

wants to merge 7 commits into from
+145 −3

Conversation

richardblack-Harness
Copy link
Collaborator

@richardblack-Harness richardblack-Harness commented Dec 23, 2024 •
edited
Loading

Description

  • Add new AH Troubleshooting doc section, as well as horizontal sidebar items for various topics depending on usage stage:
  • New to Artifact Registry?
  • Use Artifact Registry
  • Troubleshooting

PRs must meet these requirements to be merged:

  • Successful preview build.
  • Code owner review.
  • No merge conflicts.
  • Release notes/new features docs: Feature/version released to at least one prod environment.

@richardblack-Harness richardblack-Harness added DO NOT MERGE PR is not ready to be merged. Content - gap / tech debt Content flag: This PR includes missing content or docs tech debt labels Dec 23, 2024
@richardblack-Harness richardblack-Harness self-assigned this Dec 23, 2024
@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67696351d3b34058eebb7e4b--harness-developer.netlify.app

@richardblack-Harness
Copy link
Collaborator Author

@lenasano would you mind reviewing this, please?
It's nearly there, but I noticed I've lost the Artifact Registry landing page component at the top of the sidebar.
CleanShot 2024-12-23 at 13 47 15

@richardblack-Harness richardblack-Harness marked this pull request as draft December 23, 2024 13:50
@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67696bb743e3a053c34820ea--harness-developer.netlify.app

@lenasano lenasano force-pushed the AH-809/troubleshooting branch from c784da7 to 453a078 Compare December 23, 2024 14:34
@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://676975c762c0552d19c6ce9c--harness-developer.netlify.app

@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://676976b2a7940233bc47c124--harness-developer.netlify.app

@richardblack-Harness richardblack-Harness marked this pull request as ready for review December 23, 2024 14:47
@bot-gitexp-user
Copy link

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://67697ceca794023c2f47c0b1--harness-developer.netlify.app

wei-harness
wei-harness requested changes Dec 23, 2024
View reviewed changes
Copy link
Collaborator

@wei-harness wei-harness left a comment

Choose a reason for hiding this comment

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

Sidebar is missing on AR landing page:
https://67696351d3b34058eebb7e4b--harness-developer.netlify.app/docs/artifact-registry
Screenshot 2024-12-23 at 9 42 54 AM

@richardblack-Harness
Copy link
Collaborator Author

Morning @wei-harness, the build you referenced is an earlier one. This issue was fixed in newer commits.
Here is the latest build: https://67697ceca794023c2f47c0b1--harness-developer.netlify.app/

@richardblack-Harness richardblack-Harness removed the DO NOT MERGE PR is not ready to be merged. label Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wei-harness wei-harness wei-harness approved these changes

@SushrutHarness SushrutHarness Awaiting requested review from SushrutHarness SushrutHarness is a code owner

@shankaranandh shankaranandh Awaiting requested review from shankaranandh shankaranandh is a code owner

@shivagowda shivagowda Awaiting requested review from shivagowda shivagowda is a code owner

@rohanmaharjan100 rohanmaharjan100 Awaiting requested review from rohanmaharjan100 rohanmaharjan100 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@richardblack-Harness richardblack-Harness

Labels
Content - gap / tech debt Content flag: This PR includes missing content or docs tech debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@richardblack-Harness @bot-gitexp-user @wei-harness @lenasano