Skip to content

Conversation

javabster
Copy link
Contributor

This PR replaces any qiskit.org links with 1XP or github equivalents. Need to wait to merge until I can update the deprecation policy and maintainer guide links after Qiskit/qiskit#11218 merges.

This should partially resolve #297 , however for the api links we'll need to make updates for those directly in the repos where the API refs live (maybe this is beyond the scope of the original issue?)

@Eric-Arellano
Copy link
Collaborator

however for the api links we'll need to make updates for those directly in the repos where the API refs live (maybe this is beyond the scope of the original issue?)

I intended it to be in scope. But we could create a follow up issue if we want?

@javabster
Copy link
Contributor Author

javabster commented Nov 20, 2023

I intended it to be in scope. But we could create a follow up issue if we want?

Yeah, like with other API updates I think first and foremost this should be the responsibility of the devs that create the API reference. At least for now as our workload is so high as is. Perhaps we could open issues in the other repos?

@Eric-Arellano
Copy link
Collaborator

like with other API updates I think first and foremost this should be the responsibility of the devs that create the API reference.

I'm fine with that in general in the future for long-term maintenance. But as the team responsible for the qiskit.org migration, I do think it's appropriate for our team to do.

It should be fine if this slips from https://github.com/Qiskit/documentation/milestone/3 to https://github.com/Qiskit/documentation/milestone/5 thanks to redirects.

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.

thank you!

- Experiment with the [migration tutorial.](https://qiskit.org/ecosystem/ibm-provider/tutorials/Migration_Guide_from_qiskit-ibmq-provider.html)
- Experiment with the [migration tutorial.](./qiskit-runtime)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a reference to itself

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah ok, maybe I should just remove this link entirely then, ad that old provider url has been replaced with this page

Co-authored-by: Eric Arellano <14852634+Eric-Arellano@users.noreply.github.com>
@Eric-Arellano Eric-Arellano added this pull request to the merge queue Nov 22, 2023
Merged via the queue into main with commit ca4f0dd Nov 22, 2023
@Eric-Arellano Eric-Arellano deleted the am/remove-qiskit.org branch November 22, 2023 15:54
frankharkins pushed a commit to frankharkins/documentation that referenced this pull request Jul 22, 2024
This PR replaces any qiskit.org links with 1XP or github equivalents.
Need to wait to merge until I can update the deprecation policy and
maintainer guide links after
Qiskit/qiskit#11218 merges.

This should partially resolve Qiskit#297 , however for the api links we'll
need to make updates for those directly in the repos where the API refs
live (maybe this is beyond the scope of the original issue?)

---------

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
None yet
Development

Successfully merging this pull request may close these issues.

Remove links to qiskit.org from API docs
2 participants