Skip to content

Feat/swap rss locations/126#130

Merged
StefanS-O merged 3 commits intoJupiterBroadcasting:mainfrom
elreydetoda:feat/swap_rss_locations/126
Jul 20, 2022
Merged

Feat/swap rss locations/126#130
StefanS-O merged 3 commits intoJupiterBroadcasting:mainfrom
elreydetoda:feat/swap_rss_locations/126

Conversation

@elreydetoda
Copy link
Copy Markdown
Collaborator

@elreydetoda elreydetoda commented Jul 20, 2022

closes #126

I did the following:

  • removed the rss build step in the config.toml (for the main JB site only)
    • this alone actually reduced my build time immensely! So, even though hugo can do it it's probably good (from a scalability perspective) that we're looking to do it in other ways.
      image
  • in the html code I've left all the code in that @StefanS-O used (didn't even touch the list.rss.xml file), but I did
    • a hugo comment {{/* */}} to prevent it from trying to call (it used a with though so it would've been fine either way)
    • then commented out the HTML code inside the template
  • Added the 2 All shows RSS feeds to the config.toml (audio + video)
  • Only am calling the audio all shows feed in the meta.html page though

Let me know if there is anything that I missed 😁

@kbondarev kbondarev requested a review from StefanS-O July 20, 2022 14:26
@StefanS-O StefanS-O merged commit 2607d87 into JupiterBroadcasting:main Jul 20, 2022
@elreydetoda elreydetoda deleted the feat/swap_rss_locations/126 branch July 20, 2022 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RSS feed references - point to Fireside feeds

2 participants