Skip to content

Add "Get farmOS, "Develop farmOS", and "Follow farmOS" headers and links to homepage#87

Merged
mstenta merged 3 commits into
farmOS:mainfrom
mstenta:homepage-links
Jan 30, 2025
Merged

Add "Get farmOS, "Develop farmOS", and "Follow farmOS" headers and links to homepage#87
mstenta merged 3 commits into
farmOS:mainfrom
mstenta:homepage-links

Conversation

@mstenta

@mstenta mstenta commented Jan 15, 2024

Copy link
Copy Markdown
Member

This add a bunch of links to the homepage, based on feedback we've heard from the community. It also adds 3 <h2> headers to organize everything:

  • Get farmOS
  • Develop farmOS
  • Follow farmOS

@netlify

netlify Bot commented Jan 15, 2024

Copy link
Copy Markdown

Deploy Preview for gracious-brattain-bdd606 ready!

Name Link
🔨 Latest commit 3c9f784
🔍 Latest deploy log https://app.netlify.com/sites/gracious-brattain-bdd606/deploys/679b84f421581f000877d9ab
😎 Deploy Preview https://deploy-preview-87--gracious-brattain-bdd606.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@wotnak

wotnak commented Jan 15, 2024

Copy link
Copy Markdown
Collaborator

Looks good, although I'm not sure if the repositories list is really needed on the homepage. It pushes the 'Follow farmOS' section down, and Github org with all repos is already linked right above it. Maybe a better place to list important git repos would be in the Github org README.

@mstenta

mstenta commented Jan 15, 2024

Copy link
Copy Markdown
Member Author

For what it's worth, part of the motivation for adding the repositories was related to: #22 (comment)

@mstenta

mstenta commented Jan 15, 2024

Copy link
Copy Markdown
Member Author

I agree that visitors can find the repositories if they need to, but another reason to include them on the homepage (IMO), is for emphasis to non-developers. I think there is a positive psychological effect to a farmer or other non-developer visiting farmOS.org for the first time to see the list of code repos, which gives a sense that this is a "real" project with multiple pieces that many people are working on. I don't have anything to base that on, just a sense... :-)

If there's consensus that it isn't necessary or is detrimental to include them, I'm happy to remove that commit and we can find another place to put direct links to repos in farmOS.org.

@mstenta

mstenta commented Jan 30, 2025

Copy link
Copy Markdown
Member Author

Forgot about this PR!

Looking at it with fresh eyes, I agree with @wotnak's comment, so I removed the individual repository links.

Rebased and force pushed. Ready for review again. :-)

Comment thread src/content/index.md
## Develop farmOS

To join the farmOS community development efforts, read the
[Contributing to farmOS] overview, and join the community forum, chat, or calls.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

there's no url set for this link
image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks @wotnak! Fixed and force pushed.

@mstenta mstenta requested a review from wotnak January 30, 2025 13:56
@mstenta mstenta merged commit 06f2c51 into farmOS:main Jan 30, 2025
@mstenta mstenta deleted the homepage-links branch January 30, 2025 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants