-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Blog #49
Conversation
✅ Deploy Preview for gracious-brattain-bdd606 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
This looks GREAT @jgaehring!!! Can we include the year and month in the URLs that are generated? eg: I also wonder if we could go with 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":
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!! 😍 |
Thanks, @mstenta! I just drafted up some more detailed notes on the forum, but I can address a few of your points here.
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
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
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. |
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: |
@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. |
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.