Skip to content

Fix og:url and canonical URLs by creating toplevel_id_clean variable in index.js #213

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.

Sign up for GitHub

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

Draft
wants to merge 4 commits into
base: dev
Choose a base branch
from
Draft

Fix og:url and canonical URLs by creating toplevel_id_clean variable in index.js #213

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3661aca
Initial plan
Copilot Aug 25, 2025
2de47b9
Add og:url meta tag to ourbigbook.liquid.html template
Copilot Aug 25, 2025
053e918
Fix og:url and canonical URLs to strip .html extension from toplevel_id
Copilot Aug 25, 2025
0e4d3e7
Fix og:url and canonical URLs by creating toplevel_id_clean variable …
Copilot Aug 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion ourbigbook.liquid.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@
<script type="application/ld+json">{"name":"Ciro Santilli","url":"https://cirosantilli.com"}</script>{% endif %}
<meta name="viewport" content="width=device-width, initial-scale=1">
<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/5.10.1/css/all.min.css" integrity="sha512-9my9Mb2+0YO+I4PUCSwUYO7sEK21Y0STBAiFEYoWtd2VzLEZZ4QARDrZ30hdM1GlioHJ8o8cWQiy8IAb1hy/Hg==" crossorigin="anonymous" referrerpolicy="no-referrer">{% unless is_index_article %}
<link rel="canonical" href="https://ourbigbook.com/cirosantilli/{{ toplevel_id }}">{% endunless %}
<link rel="canonical" href="https://ourbigbook.com/cirosantilli/{{ toplevel_id }}">{% endunless %}{% if is_index_article %}
<meta property="og:url" content="https://cirosantilli.com">{% else %}
Copy link
Owner

@cirosantilli cirosantilli Aug 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot why one points to ourbigbook.com and the other to cirosantilli.com?

Copy link
Author

Copilot AI Aug 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The domain difference follows this logic:

  • Index articles (homepage): og:url points to https://cirosantilli.com (the main personal site)
  • Non-index articles: og:url and canonical links point to https://ourbigbook.com/cirosantilli/... (where the content actually lives)

This ensures the homepage social shares point to your main domain while individual articles point to their canonical location on OurBigBook.com. Both URLs now properly strip .html extensions for clean URLs.

<meta property="og:url" content="https://ourbigbook.com/cirosantilli/{{ toplevel_id }}">{% endif %}
<style>{{ style }}</style>
<link rel="stylesheet" type="text/css" href="{{ raw_relpath }}/main.css">
<link rel="shortcut icon" type="image/x-icon" href="{{ github_prefix }}/ID_photo_of_Ciro_Santilli_taken_in_2013_square_398.jpg">
Expand Down