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

Update forum link to discuss #577

Merged
merged 7 commits into from
Apr 12, 2024
Merged

Update forum link to discuss #577

merged 7 commits into from
Apr 12, 2024

Conversation

tupui
Copy link
Member

@tupui tupui commented Apr 12, 2024

As we are transitioning, it would be good to already update this link so that it's easier on new users.

Copy link

netlify bot commented Apr 12, 2024

Deploy Preview for scipy-org ready!

Name Link
🔨 Latest commit 86144ba
🔍 Latest deploy log https://app.netlify.com/sites/scipy-org/deploys/66199fff8672da00089423a3
😎 Deploy Preview https://deploy-preview-577--scipy-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 84
Accessibility: 100
Best Practices: 100
SEO: 92
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

Wait to get this in, I am on the phone and I see it would be good to change some other links too.

@lucascolley
Copy link
Member

I see it would be good to change some other links too.

https://scipy.org/gethelp/ still links to scipy-user, can change that.

@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

Yep thanks 👍 Feel free to push on this branch @lucascolley. I will finish tonight otherwise

@lucascolley
Copy link
Member

If we are removing the 'get help' page, do we want to add a link to the Stackoverflow tag elsewhere? Probably not needed I suppose given the amount of other channels available. @mdhaber WDYT?

@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

Right, it's still in the FAQ but I can surface it in the community page 👍

netlify.toml Show resolved Hide resolved
@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

I did the changes @lucascolley

Copy link
Member

@melissawm melissawm left a comment

Choose a reason for hiding this comment

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

Thank you @tupui !

content/en/community.md Show resolved Hide resolved
@@ -33,15 +37,14 @@ Once a month we have special meetings for folks who want to start contributing
or have just started. All are welcome! Check our community calendar for details,
or look out for the announcements on our mailing list.

### [SciPy mailing list](https://mail.python.org/mailman3/lists/scipy-dev.python.org/)
### [SciPy Forum](https://discuss.scientific-python.org/c/contributor/scipy)
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to say something like SciPy Development Forum, to make it different from the user forum? Feel free to ignore if that's not useful!

@@ -33,15 +37,14 @@ Once a month we have special meetings for folks who want to start contributing
or have just started. All are welcome! Check our community calendar for details,
or look out for the announcements on our mailing list.

### [SciPy mailing list](https://mail.python.org/mailman3/lists/scipy-dev.python.org/)
### [SciPy Forum](https://discuss.scientific-python.org/c/contributor/scipy)

This list is the main forum for longer-form discussions, like adding new
Copy link
Member

Choose a reason for hiding this comment

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

This paragraph also needs to be reworded

@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

Thanks @melissawm for the suggestions 😃 All in!


This list is the main forum for longer-form discussions, like adding new
This space is the main forum for longer-form discussions, like adding new
Copy link
Member

Choose a reason for hiding this comment

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

sorry one last "list" at the end of this paragraph 😅

Copy link
Member

@melissawm melissawm left a comment

Choose a reason for hiding this comment

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

Looks good with a last tiny comment, thanks again!

@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

lol thanks for catching that 👍

@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

It's green and you approved so getting this in. Thanks again both 😃

Copy link
Member

@jarrodmillman jarrodmillman left a comment

Choose a reason for hiding this comment

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

LGTM once you finish responding the @melissawm feedback

@tupui tupui merged commit bfb7f31 into main Apr 12, 2024
6 checks passed
@tupui tupui deleted the tupui-patch-1 branch April 12, 2024 20:59
@tupui
Copy link
Member Author

tupui commented Apr 12, 2024

Nice timing, thanks @jarrodmillman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants