Skip to content

Conversation

0xBora
Copy link
Collaborator

@0xBora 0xBora commented Jul 7, 2025

Checklist

Updating documentation or Bugfix

Fixes build warnings regarding to blogs

Changes Made

  • Fix Blog Tags

11 tag warnings affecting SEO and categorization
Fix case sensitivity (AI → ai, DeFi → defi) and add missing media tag
Files: blog/tags.yml + affected blog posts

  • Add Blog Truncation Markers

8 posts without preview truncation
Add markers to recent posts
Files: Blog posts from 2025

RATIONALE: Related to #1587

rphair
rphair previously approved these changes Jul 7, 2025
Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

nice... I checked in local build ✅ and it has't been this clean in a while! Only warnings still remaining are:

  • [WARNING] Docusaurus found broken links! ...
  • [WARNING] Docusaurus found broken anchors! ...

@0xBora
Copy link
Collaborator Author

0xBora commented Jul 7, 2025

@rphair will be addresssing them in the following PR too in 5-10 minutes :)

@rphair rphair dismissed their stale review July 10, 2025 19:45

withholding review pending style features: canonical lowercase + no truncate video pages

@katomm
Copy link
Member

katomm commented Jul 17, 2025

We need some documentation on https://developers.cardano.org/docs/portal-contribute for this

@0xBora
Copy link
Collaborator Author

0xBora commented Jul 25, 2025

@katomm I went ahead and fixed both issues. All uppercase tags are now lowercase in the blog posts and tags.yml and the markers are removed from the Developer Office Hours posts, as we will intentionally avoid using them.

I added some contribution guidelines to /docs/portal-contribute.md that explain the tag naming conventions

@0xBora 0xBora requested review from katomm and rphair July 25, 2025 23:43
Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

I checked in local build ✅ and everything looks & works great: blog is at least as good as before, with clear explanations of tags & truncation at /docs/portal-contribute#blog-contributions.

One improvement for contributors (and probably to keep our CI logs clear as well) would be to add onUntruncatedBlogPosts: 'ignore' to the blog plugin; seen in the build process:

[WARNING] Docusaurus found blog posts without truncation markers:
...
We recommend using truncation markers (`<!-- truncate -->` or `{/* truncate */}`) in blog posts to create shorter previews on blog paginated lists.
Tip: turn this security off with the `onUntruncatedBlogPosts: 'ignore'` blog plugin option.

@0xBora
Copy link
Collaborator Author

0xBora commented Jul 26, 2025

Added onUntruncatedBlogPosts: 'ignore' as @rphair mentioned to blog plugin config and fixed tag casing mismatch (Open sourceopen source.

Build warnings should now be cleaner.

@0xBora 0xBora requested a review from rphair July 26, 2025 17:09
Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

Definitely cleaner 🤓 ... only 2 warnings I see in local build now are (outside the scope of this PR):


This one has been around for a while:

[WARNING] The image at "/home/rphair/archive/build/developer-portal/static/img/nfts/overview_nfts.svg" can't be read correctly. Please ensure it's a valid image.
unsupported file type: undefined (file: /home/rphair/archive/build/developer-portal/static/img/nfts/overview_nfts.svg)
[WARNING] The image at "/home/rphair/archive/build/developer-portal/static/img/nfts/overview_nfts.svg" can't be read correctly. Please ensure it's a valid image.
unsupported file type: undefined (file: /home/rphair/archive/build/developer-portal/static/img/nfts/overview_nfts.svg)

Needs periodic tune-up, as last we did here (to my knowledge):

Browserslist: browsers data (caniuse-lite) is 8 months old. Please run:
  npx update-browserslist-db@latest
  Why you should do it regularly: https://github.com/browserslist/update-db#readme

p.s. Fixed by #1610.

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.

5 participants