-
Notifications
You must be signed in to change notification settings - Fork 39
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
Broken breadcrumb in OpenShift docs #169
Comments
Changes required on the OpenShift side:
|
I have replicated the issue with my local asciibinder build. Using @gabriel-rh 's container image with his updated version of asciibinder, I verified the fixes worked as expected. |
There is a bit of a chicken-and-egg problem releasing this fix. The new asciibinder (1.0.2) will work with the old templates, but the old asciibinder (1.0.1 or 1.0) will fail against the new templates (see error below). That means that the new asciibinder needs to be released before the template changes are merged (otherwise the build for the templates PR will fail). But then, after merging the new templates, if writers do not upgrade to the latest version of asciibinder, their local builds will fail.
|
Sample content: https://github.com/gabriel-ocp/ocp-breadcrumbs
Sample output: https://gabriel-ocp.github.io/ocp-breadcrumbs/openshift-enterprise/main/level1/level2/level3/level3-intro.html
Note how the level 2 breadcrumb disappears when you navigate to the 3rd level
===================
Originally reported in:
https://issues.redhat.com/browse/OADP-443
From https://docs.openshift.com/container-platform/4.9/backup_and_restore/application_backup_and_restore/installing/installing-oadp-aws.html
You can click on Backup and Restore (see screenshot) and be linked to a broken https://docs.openshift.com/container-platform/4.9/backup_and_restore/application_backup_and_restore/backup_and_restore/index.html
The text was updated successfully, but these errors were encountered: