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

Change index entry from patterns _toc.json #1484

Merged
merged 5 commits into from
Jun 4, 2024

Conversation

arnaucasau
Copy link
Collaborator

This PR changes the patterns script to use '"url": "/guides"' instead of "url": "/guides/index"' for the index entry of the guides _toc.json. The PR doesn't modify the entries.py script to avoid having an empty slug.

@arnaucasau arnaucasau requested a review from Eric-Arellano June 3, 2024 16:49
@qiskit-bot
Copy link
Contributor

Thanks for contributing to Qiskit documentation!

Before your PR can be merged, it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. Thanks! 🙌

Copy link
Collaborator

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

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

Thanks! Can you please run the script to confirm that the redirects file doesn't change at all? You'll need to rm -rf docs/guides after

scripts/patterns-reorg/models.py Outdated Show resolved Hide resolved
scripts/patterns-reorg/models.py Outdated Show resolved Hide resolved
scripts/patterns-reorg/models.py Outdated Show resolved Hide resolved
arnaucasau and others added 3 commits June 4, 2024 17:34
Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
@arnaucasau
Copy link
Collaborator Author

Checked, the script only created the docs/guides folder and it doesn't change the redirects file 👍

@Eric-Arellano Eric-Arellano added this pull request to the merge queue Jun 4, 2024
Merged via the queue into Qiskit:main with commit 3025b7c Jun 4, 2024
2 checks passed
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
This PR changes the patterns script to use '`"url": "/guides"`' instead
of `"url": "/guides/index"'` for the index entry of the guides
`_toc.json`. The PR doesn't modify the `entries.py` script to avoid
having an empty slug.

---------

Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants