Replace unused Bundler Slack links with RubyGems Discussions link#1635
Replace unused Bundler Slack links with RubyGems Discussions link#1635
Conversation
There was a problem hiding this comment.
Pull request overview
Updates Bundler’s homepage communication links to direct users to GitHub Discussions instead of the previously used (now unused) Bundler Slack invite link, addressing bundler-site issue #888.
Changes:
- Replaces the
join_slack_linkvariable with adiscussions_linkvariable. - Updates homepage CTAs and “Get involved” text/buttons to point to GitHub Discussions instead of Slack.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -1,4 +1,4 @@ | |||
| - join_slack_link = 'https://join.slack.com/t/bundler/shared_invite/zt-3kza12u0j-Fw3rwvPi6zqe6EC3RCDGMA' | |||
| - discussions_link = 'https://github.com/ruby/rubygems/discussions' | |||
There was a problem hiding this comment.
The new Discussions URL points to github.com/ruby/rubygems, which doesn’t match the rest of the site’s GitHub links (they consistently use github.com/rubygems/rubygems) and is likely a broken/incorrect target. Consider updating this to the rubygems/rubygems Discussions URL, and (per issue #888) linking directly to the Bundler-labeled discussions query so users land in the right place.
| - discussions_link = 'https://github.com/ruby/rubygems/discussions' | |
| - discussions_link = 'https://github.com/rubygems/rubygems/discussions?discussions_q=label%3ABundler' |
| .my-4.large-font | ||
| Bundler has a lot of contributors and users, and we would love to have your help! | ||
| If you have questions, join #{link_to 'the Bundler Slack', join_slack_link} | ||
| If you have questions, join #{link_to 'the RubyGems Discussions', discussions_link} |
There was a problem hiding this comment.
Copy says “join” RubyGems Discussions, but GitHub Discussions isn’t something users typically “join” (unlike Slack). Consider changing the CTA text to something like “visit”, “ask on”, or “start a discussion” to avoid misleading guidance.
Fixed #888