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

Blog #49

Merged
merged 19 commits into from
Oct 31, 2022
Merged

Blog #49

merged 19 commits into from
Oct 31, 2022

Conversation

jgaehring
Copy link
Member

This and #48 should fulfill the implementation requirements for the first deliverable of the Work Proposal to support community content on farmos.org, which is being funded by the farmOS Open Collective. This PR should also generate a preview deployment on Netlify (the second deliverable), which is sourcing a test repository at jgaehring/farmOS-community-blog (the third deliverable), so that the full feature set can be viewed and tested.

It probably doesn't make sense to merge and deploy this, however, until there is some content ready to be published.

@netlify
Copy link

netlify bot commented Jul 1, 2022

Deploy Preview for gracious-brattain-bdd606 ready!

Name Link
🔨 Latest commit 0f2d586
🔍 Latest deploy log https://app.netlify.com/sites/gracious-brattain-bdd606/deploys/636029fcff3e0c0008fac78c
😎 Deploy Preview https://deploy-preview-49--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 settings.

@mstenta
Copy link
Member

mstenta commented Jul 1, 2022

This looks GREAT @jgaehring!!!

Can we include the year and month in the URLs that are generated? eg: /blog/2022/06/another-test/ Or is that just controlled by the directory structure of the blog markdown files in the source repo?

I also wonder if we could go with /posts/... instead of /blog/...? Just personal preference - what do others think? (And again maybe this is just a decision about how we structure the directories in the source repo?)

Lastly, does this support tags/categories yet? Or is that a next step? In the forum you mentioned that simply allowing for categories is "fairly easy":

In a technical sense, we could fairly easily create a category property in Front matter, we would just need to decide how to present those separate categories in the navigation and other UI elements.

https://farmos.discourse.group/t/work-proposal-support-for-community-content-on-farmos-org/1265/3

I think it would be great to at least start with the ability to categorize posts, and maybe just display them under the date/author line. Using them in navigation and other UI could be a next step...

LOVE to see this come together! And I can't wait to contribute blog posts!! 😍

@jgaehring
Copy link
Member Author

Thanks, @mstenta! I just drafted up some more detailed notes on the forum, but I can address a few of your points here.

Can we include the year and month in the URLs that are generated? eg: /blog/2022/06/another-test/ Or is that just controlled by the directory structure of the blog markdown files in the source repo?

Yes and yes. I've opted for parsimony wherever I can, so the URL can be determined either by the directory structure, which can be as flat or nested as desired, or by the slug frontmatter if one is provided. Although I realize it would be nice to combine those in some way, so you could use the file path and the slug together, just replacing the last path segment with the slug. But I also assumed a pretty flat directory structure to start, until a clearer need emerges to organize it more strictly. I suppose we could also introduce permalinks somehow, based on a more deterministic naming convention. Personally, I prefer URL's that are short and descriptive, but perhaps that could be varied depending on the specific category, or determined by whoever elects to take ownership of the content repository (or repositories).

I also wonder if we could go with /posts/... instead of /blog/...? Just personal preference - what do others think? (And again maybe this is just a decision about how we structure the directories in the source repo?)

Yea, I almost did this, actually, and I mentioned this on the last dev call too. Ultimately, I'm wary of being too committed to anything early on, until we have a larger corpus of posts, but all we need to do is change the baseURI in site-data.js.

Lastly, does this support tags/categories yet? Or is that a next step?

Not yet, but it is easy to add. I detailed just how that can be done over on the forum, but again, I'm trying not to put the cart before the horse, because even once we have our first post that's ready to publish, a category of one is really no category at all. Better to put energy into content creation, imo, at least for now.

@mstenta
Copy link
Member

mstenta commented Oct 31, 2022

Update: we moved @jgaehring's repo to the farmOS org: https://github.com/farmOS/farmOS-community-blog

And merged @symbioquine's first post: farmOS/farmOS-community-blog#1

Next step is to test this again and it!

I'm also creating a few next step followup issues:

@jgaehring
Copy link
Member Author

@mstenta, I just added you as a reviewer to give the final go-ahead (which you've already done before I can post this, haha) but since @symbioquine's commits from jgaehring#1 have been added to this one, and your own farmOS/farmOS-community-blog#6 was merged, I will go ahead and merge this one.

As we discussed in Element, right now blog posts will only be displayed on the main index page, and in simple reverse chronological order, rather than being added to the navigation menu. This should suffice until there is enough content to require more robust filtering and/or search features.

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.

3 participants