Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

Don't link to StackOverflow #211

Merged
merged 6 commits into from
Mar 27, 2023
Merged

Conversation

Eric-Arellano
Copy link
Contributor

@Eric-Arellano Eric-Arellano commented Mar 14, 2023

Changes

Closes #210.

Idea comes from @1ucian0

Implementation details

Uses the short link https://qisk.it/support rather than https://github.com/Qiskit/feedback/wiki/Communication-Channels-&-Support because it is more stable, e.g. if we move that page away from GitHub Wiki.

Also removes Slack because it's already covered by the Support Channels link.

@changeset-bot
Copy link

changeset-bot bot commented Mar 14, 2023

🦋 Changeset detected

Latest commit: bcbcf0a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@qiskit/web-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Eric-Arellano
Copy link
Contributor Author

Re: the changeset, should I be adding that? @1ucian0 perhaps how urgent is releasing a new version of this web component?

@1ucian0
Copy link
Member

1ucian0 commented Mar 14, 2023

Important, but not that urgent.

Copy link
Contributor

@HuangJunye HuangJunye left a comment

Choose a reason for hiding this comment

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

I have a small comments about whether we should keep Slack on the menu or remove it to direct all people to the support channels page which contain more information about Slack and which channels to join.

Comment on lines -189 to +190
label: 'Stack Exchange',
url: 'https://quantumcomputing.stackexchange.com/questions/tagged/qiskit',
label: 'Support Channels',
url: 'https://qisk.it/support',
Copy link
Contributor

Choose a reason for hiding this comment

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

Support channels page contain information about Slack as well. Should we remove the Slack link in this drop down menu?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I noticed that too. I think it depends how much we want people going to Slack? For Pantsbuild, that was a huge project priority so we referenced Slack everywhere like blog posts. Even if it was repetitive like this here would be.

For Qiskit, how much do want to specifically funnel people to Slack? If we want to, then I think it's worth repeating. Otherwise, we could probably remove.

The benefit of removing would be less text in the top nav bar, I suppose?

Copy link
Contributor

Choose a reason for hiding this comment

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

For me the benefits of removing it is so that people will read the information about different slack channels and where to get help for specific issues, similar to the spirit of this PR trying to guide people to read about what kind of questions should go to StackExchange instead of sending people there directly.

@HuangJunye
Copy link
Contributor

FYI, I created this an unrelated issue about removing Qiskit Textbook from the "Help and Learning" column: #214. That issue may need some more discussions. But if we reach consensus before this was merged. We can potentially address that issue in this PR if that helps to be reviewed and merged quicker.

@Eric-Arellano
Copy link
Contributor Author

Bump @abdonrd when you get a chance

@abdonrd
Copy link
Member

abdonrd commented Mar 27, 2023

Bump @abdonrd when you get a chance

You want to merge the PR? If you confirm it, I'll merge it!

Anyway, I am not actively working on this project. Better to ping @y4izus @techtolentino @eddybrando

@Eric-Arellano
Copy link
Contributor Author

Yes, please, to merging. All ready to go :)

Ah, thanks for the FYI!

@abdonrd abdonrd merged commit a5c1825 into Qiskit:main Mar 27, 2023
@Eric-Arellano Eric-Arellano deleted the stack-overflow branch March 27, 2023 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Have a general "support channels" instead of QCSE direct link
4 participants