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

Adding Slack link #815

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

d-spencer-strickland
Copy link

This adds a link to the C++ Slack workspace.

Tested under all breakpoints for various screen resolutions so it should render fairly well, but we can tweak it if there are any glitches.

@glenfe
Copy link
Member

glenfe commented Feb 16, 2024

Hi Spencer, copying my reply to you on the Boost mailing list:

  1. The place for these links (e.g. IRC and others) would be the Community page, not the main page.
    This: https://www.boost.org/community/index.html

  2. Shouldn't this say "Visit the Boost channel at the C++ Slack" or something similar?
    i.e. You want to point people to the Boost space in a particular server.

@d-spencer-strickland
Copy link
Author

@glenfe Thanks for the notes, I'll push an update by tomorrow to address. (sorry for the delay, want to keep this strictly RE: the work but have been dealing with a family emergency and want to note that since I hadn't had a chance to respond on this yet)

It seems like it will work best on the line that starts "If you are looking for more lively interaction..." and then also as a standalone link on the right nav either above or below the IRC link. If you have a suggestion on the ordering, please let me know. I am not sure which sees the most engagement from the community yet and will start following things more so that I'll have a better sense of things like that going forward. But will plan to push what makes sense to me now and can wait for any feedback. Thanks!

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.

2 participants