Refactor blog section (/blog/)#253
Open
Gijsreyn wants to merge 3 commits into
Open
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThe PR reorganizes how blog posts are discovered and displayed by introducing a ChangesBlog categorization and TOC enhancement for article discovery
🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 5
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
layouts/blog/list.html (1)
10-15:⚠️ Potential issue | 🟠 Major | 🏗️ Heavy liftLocalize the new list-page UI copy via i18n keys.
The new section titles, descriptions, CTA text, and action labels are hardcoded. Please move these literals to localization keys and update translation files alongside this template change.
As per coding guidelines, "Localize all strings using string keys; remove any orphaned string keys."
Also applies to: 38-40, 77-79, 98-99, 117-119, 138-139, 152-154, 164-171
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@layouts/blog/list.html` around lines 10 - 15, The template hardcodes UI copy (e.g., "Knowledge Base", the .hero-title "Blog Posts", .hero-description paragraph and other CTA/action labels referenced at lines 38-40, 77-79, 98-99, 117-119, 138-139, 152-154, 164-171); replace each literal with an i18n key lookup using the project's template translation helper (same pattern as other localized templates), add corresponding keys to the translation files for all supported locales, and remove/rename any orphaned string keys so keys and template usage remain synchronized.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@assets/js/main.js`:
- Around line 368-370: initTocHighlight currently resolves TOC targets with
document.querySelector(link.getAttribute("href")) and then assumes headings[0]
and current.id exist; fix by robustly resolving hrefs: for hash-only hrefs use
the id lookup path (extract href.slice(1), decodeURIComponent and call
document.getElementById) to avoid CSS selector escaping issues, and for other
hrefs fall back to querySelector inside a try/catch to ignore invalid selectors;
then filter out falsy results into headings and guard uses of headings[0] and
current (e.g., check headings.length > 0 and current != null before accessing
current.id) so no throws occur when headings is empty or selectors are invalid
(referencing initTocHighlight, tocLinks, headings, and current.id).
In `@content/blog/create-an-article.en.md`:
- Line 6: Add a new entry to CHANGELOG.md under the "Unreleased" section
documenting the blog metadata change (dsc_family: "Community") and the updated
TOC behavior introduced by this PR; mention the affected file
content/blog/create-an-article.en.md and briefly summarize the change (metadata
addition and TOC behavior change) so the Unreleased section records the PR
impact.
In `@layouts/blog/list.html`:
- Around line 109-123: Merged collection $generalPages (created by append
$communityPages $uncategorized) is not re-sorted so posts from both sources
aren’t in global chronological order; after you create $generalPages, run Hugo’s
sort on the Date (e.g., sort $generalPages "Date" "desc") and use that sorted
value in the subsequent range. Locate the $generalPages assignment and the range
that iterates it, replace the range source with the sorted collection (or
reassign $generalPages to the sorted result) so items are globally ordered by
Date descending.
In `@layouts/blog/single.html`:
- Around line 8-11: The "Back to Blog" anchor and other hardcoded UI texts
(e.g., the anchor with class "btn btn-ghost" containing the literal "Back to
Blog" and the other literals noted at the same template) must be replaced with
Hugo i18n string keys and the corresponding translations added to the site's
i18n files; update the template to call the i18n helper (use a key like
"back_to_blog") instead of the literal, and add that key and translations to
your i18n JSON/TOML/YAML so the UI is translatable and consistent with the repo
policy.
In `@layouts/community/maintainers.html`:
- Around line 70-78: Wrap the resources.GetRemote call in try and check the
returned error object instead of relying on .Err on the raw call: call try
resources.GetRemote with $path and $opts, capture the result (e.g., assign to a
variable), then test that variable's .Err before proceeding to
transform.Unmarshal and set $data; ensure you still use transform.Unmarshal and
assign to $data only when the try result has no error and contains Content.
---
Outside diff comments:
In `@layouts/blog/list.html`:
- Around line 10-15: The template hardcodes UI copy (e.g., "Knowledge Base", the
.hero-title "Blog Posts", .hero-description paragraph and other CTA/action
labels referenced at lines 38-40, 77-79, 98-99, 117-119, 138-139, 152-154,
164-171); replace each literal with an i18n key lookup using the project's
template translation helper (same pattern as other localized templates), add
corresponding keys to the translation files for all supported locales, and
remove/rename any orphaned string keys so keys and template usage remain
synchronized.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 8ec9e247-f6b3-4ccc-a3c3-f24018fe0a32
📒 Files selected for processing (21)
archetypes/blog.mdassets/css/futuristic.cssassets/js/main.jsconfig.tomlcontent/blog/DSC-is-dead-long-live-dsc.en.mdcontent/blog/add-codecov-support-to-repository.mdcontent/blog/class-based-dsc-resources.mdcontent/blog/convert-a-module-for-continuous-delivery.en.mdcontent/blog/convert-master-to-main.en.mdcontent/blog/convert-tests-to-pester5-for-dsc-community-repository.mdcontent/blog/converting-tests-to-pester5.mdcontent/blog/create-an-article.en.mdcontent/blog/dsc-maintenance-windows.en.mdcontent/blog/how-to-use-dsc-logging.en.mdcontent/blog/updating-sampler-github-tasks.mdcontent/blog/use-dscresource-common-functions-in-module.en.mdcontent/blog/what-is-microsoft-dscv3.mdcontent/blog/your-first-class-based-microsoft-dsc-v3-resource.mdlayouts/blog/list.htmllayouts/blog/single.htmllayouts/community/maintainers.html
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request redesigns the
/blog/listing page and individual blog post layout by reducing clutter, adding structure (PSDSC, Microsoft DSC, and Community), and improving readability (TOC).Each section now has a distinct icon, title, and description header seen at local runtime:
The blog post is more narrow with a TOC added:
With the new frontmatter (
dsc_family), it gets controlled on which section posts are added. Lastly, there was a small template fix as this was from older Hugo versions (type: "post") e.g. Hugo's template lookup uses the type field to pick a layouts subfolder — withtype: "post"it searchedlayouts/post/(non-existent), skipped blog, and fell back tosingle.html` (which rendered the "all blog posts" sidebar). Without an explicit type, Hugo defaults to the section name (blog) and correctly resolves to single.html.This change is