Skip to content

(doc) Fixing link to Community Slack - #412

Merged
pmcmaw merged 1 commit into
mainfrom
puppet-strings-slack-link-fix
Aug 27, 2025
Merged

(doc) Fixing link to Community Slack#412
pmcmaw merged 1 commit into
mainfrom
puppet-strings-slack-link-fix

Conversation

@jst-cyr

@jst-cyr jst-cyr commented Aug 27, 2025

Copy link
Copy Markdown
Member

Summary

README updated. The URL for the community Slack has been updated to correctly point to https://puppetcommunity.slack.com/ instead of https://slack.puppet.com

@jst-cyr jst-cyr self-assigned this Aug 27, 2025
@jst-cyr
jst-cyr requested review from a team and bastelfreak as code owners August 27, 2025 20:12
@pmcmaw
pmcmaw merged commit 4da18c1 into main Aug 27, 2025
5 checks passed
@pmcmaw
pmcmaw deleted the puppet-strings-slack-link-fix branch August 27, 2025 20:28
@bastelfreak

Copy link
Copy Markdown
Collaborator

@jst-cyr @pmcmaw I don't think this change is correct. puppetcommunity.slack.com always was the login page. You cannot register there. slack.puppet.com redirected to a register page.

@jst-cyr

jst-cyr commented Aug 28, 2025

Copy link
Copy Markdown
Member Author

@bastelfreak : I agree, it's not an equivalent fix. The slack.puppet.com URL doesn't work right now, though, so I was dropping in a quick fix to get a working URL again.

On the puppet.com website, we use this: https://join.slack.com/t/puppetcommunity/shared_invite/zt-30zbgx1ti-wRR9Et_K~Nt~D62jLixWcw

That takes to a registration page and may be a better destination, but it's probably better to have that centrally managed behind a redirect instead of in every README.

@jst-cyr

jst-cyr commented Sep 5, 2025

Copy link
Copy Markdown
Member Author

@pmcmaw and @bastelfreak : I've just been notified by IT that my request to get slack.puppet.com working again as a join link has been completed. I'll submit a new PR to get us reverted back to the old URL now that it works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants