Skip to content

Commit 50fe782

Browse files
authored
Merge pull request #80 from DataTalksClub/podcast-improvements-seo
Update sitemap.xml to escape ampersands in URLs for improved XML compliance
2 parents 03bbb8f + ecd7ef2 commit 50fe782

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

Gemfile

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,18 +8,17 @@ source "https://rubygems.org"
88
# This will help ensure the proper Jekyll version is running.
99
# Happy Jekylling!
1010

11-
gem "jekyll"
11+
# Jekyll is included via github-pages gem, so we don't need a standalone jekyll gem
1212
# This is the default theme for new Jekyll sites. You may change this to anything you like.
1313
#gem "minima", "~> 2.5"
1414
gem "jekyll-theme-cayman"
1515
gem "github-pages", group: :jekyll_plugins
16+
gem "faraday-retry"
1617

1718
# This is the default theme for new Jekyll sites. You may change this to anything you like.
1819
#gem "minima", "~> 2.5"
1920

20-
# If you want to use GitHub Pages, remove the "gem "jekyll"" above and
21-
# uncomment the line below. To upgrade, run `bundle update github-pages`.
22-
# gem "github-pages", group: :jekyll_plugins
21+
# GitHub Pages is already configured above. To upgrade, run `bundle update github-pages`.
2322
# If you have any plugins, put them here!
2423
group :jekyll_plugins do
2524
gem "jekyll-feed", "~> 0.12"

_config.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
theme: jekyll-theme-cayman
22
url: https://datatalks.club
33
name: DataTalks.Club
4+
title: DataTalks.Club
45
twitter: '@DataTalksClub'
56
permalink: /blog/:title.html
67

sitemap.xml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,27 +25,27 @@
2525
</url>
2626
{% for post in site.posts %}
2727
<url>
28-
<loc>{{site.url}}{{ post.url | remove: 'index.html' }}</loc>
28+
<loc>{{site.url}}{{ post.url | remove: 'index.html' | replace: '&', '&amp;' }}</loc>
2929
</url>
3030
{% endfor %}
3131
{% for author in site.people %}
3232
<url>
33-
<loc>{{site.url}}{{ author.url }}</loc>
33+
<loc>{{site.url}}{{ author.url | replace: '&', '&amp;' }}</loc>
3434
</url>
3535
{% endfor %}
3636
{% for book in site.books %}
3737
<url>
38-
<loc>{{site.url}}{{ book.url }}</loc>
38+
<loc>{{site.url}}{{ book.url | replace: '&', '&amp;' }}</loc>
3939
</url>
4040
{% endfor %}
4141
{% for conf in site.conferences %}
4242
<url>
43-
<loc>{{site.url}}{{ conf.url }}</loc>
43+
<loc>{{site.url}}{{ conf.url | replace: '&', '&amp;' }}</loc>
4444
</url>
4545
{% endfor %}
4646
{% for p in site.podcast %}
4747
<url>
48-
<loc>{{site.url}}{{ p.url }}</loc>
48+
<loc>{{site.url}}{{ p.url | replace: '&', '&amp;' }}</loc>
4949
</url>
5050
{% endfor %}
5151

0 commit comments

Comments
 (0)