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

Fix link in readme #3504

Merged
merged 2 commits into from
Apr 16, 2023
Merged

Fix link in readme #3504

merged 2 commits into from
Apr 16, 2023

Conversation

dorner
Copy link
Collaborator

@dorner dorner commented Mar 27, 2023

Discovered in #3503 .

@dorner dorner requested a review from cielf March 27, 2023 13:35
@@ -81,7 +81,7 @@ If you're new here, here are some things you should know:

#### 💬 Join us on slack
Please feel free to join us on Slack!
You can sign up at https://rubyforgood.herokuapp.com and find us in #human-essentials.
You can sign up at https://rubyforgood.org and find us in #human-essentials.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Mayyyybe? 😁 I figured whoever wrote the original doc wanted them to go to the page first (maybe to see more info about what RfG does?) Obviously I can change if you like.

Copy link
Collaborator

@cielf cielf Mar 30, 2023

Choose a reason for hiding this comment

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

Let's check with @seanmarcia -- I think we should have a link to rubyforgood.org somewhere in the readme, but if someone is specifically wanting to get to the slack, it doesn't seem fair to make them keep searching for it.

Copy link
Member

Choose a reason for hiding this comment

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

We can link to the page with the slack link here: https://rubyforgood.org/join-us or you can just put the invite link on (that is on that page) in the readme. Whichever you think is best :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ok.. How about we do this...
1/ Use the direct link to the invite in the section on Slack

2/ Add a section below for Ruby for Good with text something like:

Ruby for Good
Human Essentials is one of many projects initiated and run by Ruby for Good. You can find out more about Ruby for Good at https://rubyforgood.org

It's plausible that it will redirect some people to other projects, but I'm ok with that if they are still Gooding.

@dorner
Copy link
Collaborator Author

dorner commented Apr 14, 2023

@cielf pushed the fixes!

@cielf
Copy link
Collaborator

cielf commented Apr 16, 2023

@dorner Looks good to me! I do see there are some more html irregularities of the same sort that you fixed - but that can be a junior task.

@cielf cielf merged commit f830327 into main Apr 16, 2023
@cielf cielf deleted the fix-link-in-readme branch April 16, 2023 23:19
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.

3 participants