Skip to content

Fix/tag routing - #104

Merged
GeauxJD merged 1 commit into
ossf:mainfrom
PaarthPandey10:fix/tag-routing
Jul 20, 2026
Merged

Fix/tag routing#104
GeauxJD merged 1 commit into
ossf:mainfrom
PaarthPandey10:fix/tag-routing

Conversation

@PaarthPandey10

Copy link
Copy Markdown
Contributor

Describe your changes

This PR resolves several issues identified during site building:

  • Tag Routing: Fixed a casing mismatch in terms.html by normalizing tag IDs and CSS classes to lowercase. This ensures the filter logic correctly matches items with their assigned tags.
  • HTML Rendering: Added | safeHTML to the summary rendering pipeline in terms.html to prevent embedded HTML tags (like <p>, <ul>, <li>) from being escaped and displayed as plain text.
  • Build Stability: Updated deprecated .Site.DisqusShortname calls (which were causing build errors in newer Hugo versions) to the safe .Site.Params.DisqusShortname syntax across all project layouts and vendored theme partials. Added an if guard to the Disqus partial to prevent build failures when disqusShortname is empty.
  • YAML Correction: Fixed a minor YAML parsing error in content/en/rule.md that was preventing successful site assembly.

Related issue number or link

resolves #66

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@GeauxJD

GeauxJD commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

@PaarthPandey10 can you adjust this pull request (and also #103 ) so that this pull request only includes the changes that address the tag routing and 103 only includes the glossary terms and spellcheck edits?

Signed-off-by: PaarthPandey10 <paarthdxb@gmail.com>
Copilot AI review requested due to automatic review settings July 20, 2026 07:03

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@GeauxJD
GeauxJD merged commit 0cb58d5 into ossf:main Jul 20, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Browse by tags not working

3 participants