-
Notifications
You must be signed in to change notification settings - Fork 61
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
Create skeleton of blog feature #310
Conversation
Signed-off-by: Matt Stratton <[email protected]>
You can poke around (row view only right now but the code is in for grid) at https://deploy-preview-310--sample-castanet.netlify.app/blog/
|
Co-authored-by: Chris Reddington <[email protected]> Co-authored-by: Matt Stratton <[email protected]>
Signed-off-by: Matt Stratton <[email protected]>
…nto add-blog-feature
Signed-off-by: Matt Stratton <[email protected]>
…nto add-blog-feature
@mattstratton - Is there anything you'd like me to pick up from the to do list to help out? :) |
I'd say take any of them - just let me know which ones so we don't try to do it at the same time :) |
Cool! I'll likely not have a chance to look before the weekend, but will drop a note as soon as I take a look :) |
@mattstratton - I'm starting work on the metadata pieces UPDATE - I have a PR (which will be merged into your add-blog-feature branch) for review on the metadata pieces, also added a .Reverse to the queries, as they were showing the oldest blog items first. |
@mattstratton - I'm also working on updating the HEAD and the subscribe to blog piece. |
That one is now done, so I believe the following 3 are left -
UPDATE: I have updated in PR #324 as well. |
this just meant I haven't really looked at the feed to make sure that all the items are correct/include all the things that a "good" blog RSS feed should have. I have to check to see what those fields/content should be; the blog feed might be fine already! |
…update HEAD on appropriate pages and update REFERENCE.md (#324)
I think we can open up a new issue for cosmetic enhancements for the individual blog posts and then this should be able to ship |
Co-authored-by: Chris Reddington <[email protected]>
Makes sense to me. In which case, I think that all the original items in the issue (aside from cosmetic enhancements) are complete? :) Thanks! |
I'll merge this and some others and cut a release! |
Great, thanks @mattstratton - It's now in action over at https://www.cloudwithchris.com/ :) This allows me to merge my own blog/site into my podcast site, and have one central place to manage everything. On a related point to that, I've made a private change to my own private fork to show a parameter of "ShortBio" if it exists in the Guest/Host file, instead of the main content. The reason for that, is for example my own bio shown here is very long, and I'd like a shorter snippet elsewhere. I've been consistent on ShortBio across guests and hosts for consistency, so can propose that as a PR if it's a useful addition.
|
Fixes #19
This is a work in progress; still need to do a bunch of stuff:
HEAD
to include blog rss instead of episode on blog-related pagesSigned-off-by: Matt Stratton [email protected]