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 docs link from gitbook to access.redhat.com #282

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

komish
Copy link
Contributor

@komish komish commented Oct 19, 2023

Cherry-picking Rose's documentation changes which are getting overwritten in the 1.6.0 release because I forgot to have her contribute it to this repository (instead of the charts repository)

@komish komish marked this pull request as ready for review October 19, 2023 16:36
@komish komish merged commit 1bd75c6 into openshift-helm-charts:main Oct 23, 2023
7 checks passed
@komish komish deleted the rose-docs branch October 23, 2023 16:11
@@ -357,7 +361,7 @@ If the vendor label in the `OWNERS` file is wrong, you can follow the [partner
connect documentation][partners] to update the `OWNERS` file.

If the directory structure (organization name) is wrong, please reach out to
[Technology Partner Success Desk][partner-success-desk].
[Technology Partner Success Desk][partner-acceleration-desk].
Copy link
Contributor

Choose a reason for hiding this comment

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

I know this PR is already merged, but should this also be changed to "Partner Acceleration Desk" ?

Copy link
Contributor

Choose a reason for hiding this comment

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

I opened #285 to fix this.

mgoerens added a commit to mgoerens/development that referenced this pull request Oct 24, 2023
This was overlooked in openshift-helm-charts#282.

Signed-off-by: Matthias Goerens <mgoerens@redhat.com>
komish pushed a commit that referenced this pull request Oct 24, 2023
This was overlooked in #282.

Signed-off-by: Matthias Goerens <mgoerens@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants