Skip to content

Conversation

@jmagak
Copy link
Member

@jmagak jmagak commented Jul 2, 2025

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s): 1.5, 1.6, 1.7

Issue: RHIDP-6150

Preview:

@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Jul 2, 2025

Copy link
Member

@hmanwani-rh hmanwani-rh left a comment

Choose a reason for hiding this comment

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

Please see my comment: #1242 (review)
In a similar way, I feel we may not be heading in the right direction with this PR.
cc @Gerry-Forde


- Your {product-short} application is installed and running.
- You have installed the Nexus Repository Manager plugin. For the installation process, see <<Installing and configuring the Nexus Repository Manager plugin>>.
- You have installed the Nexus Repository Manager plugin. For the installation process, see documentation on Installing and configuring the Nexus Repository Manager plugin.
Copy link
Member

Choose a reason for hiding this comment

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

If we’re including the statement, could you please clarify why the link is being removed?

.Prerequisites

* Your {product-short} application is installed and running.
* You have installed the Quay plugin. For installation process, see <<installation,Installation>>.
Copy link
Member

Choose a reason for hiding this comment

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

Rather than removing the links, I’d suggest keeping them and ensuring they follow a clear structure that aligns with our migration strategy, which we may need to address in the future. Please feel free to create a Jira for the link improvement effort if needed.

@jmagak jmagak closed this Aug 5, 2025
@jmagak
Copy link
Member Author

jmagak commented Aug 5, 2025

Closing this PR after discussing with a colleague. We agreed not to fully move the prerequisite links to Additional Resources. I’ll create new PRs to address the issue with a revised approach.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants