Skip to content

GH-94: feat: Enable the multilingual post listing support in the blog page & update the application screenshots - #93

Merged
ChanukaUOJ merged 3 commits into
LDFLK:mainfrom
ChanukaUOJ:fix/application-thumbnail-update
Aug 12, 2026
Merged

GH-94: feat: Enable the multilingual post listing support in the blog page & update the application screenshots#93
ChanukaUOJ merged 3 commits into
LDFLK:mainfrom
ChanukaUOJ:fix/application-thumbnail-update

Conversation

@ChanukaUOJ

@ChanukaUOJ ChanukaUOJ commented Aug 11, 2026

Copy link
Copy Markdown
Member

This PR Closes: #94

Changes

  • Implemented a way to list the translated versions of posts in the blog page.
  • Updated the existing OpenGINXplore application screenshot in the Applications page with the newly updated one

Preview

Screenshot 2026-08-12 at 11 21 52 Screenshot 2026-08-11 at 14 19 10

Summary by CodeRabbit

New Features

  • Added language-specific links for blog posts in English, Sinhala, and Tamil where available.
  • Added translation links to the ministerial-change article in all three languages.

Improvements

  • Blog post titles now link directly to the primary translation.
  • Removed the separate “Read more” link for a cleaner article-card layout.

@ChanukaUOJ
ChanukaUOJ requested a review from zaeema-n August 11, 2026 08:48
@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a3e9e2b1-daf8-4915-aba6-7f11dac67eb3

📥 Commits

Reviewing files that changed from the base of the PR and between a1fa592 and 18e151c.

📒 Files selected for processing (2)
  • _data/posts.yml
  • _includes/blog.html
🚧 Files skipped from review as they are similar to previous changes (2)
  • _includes/blog.html
  • _data/posts.yml

📝 Walkthrough

Walkthrough

Blog post URLs now use ordered translation entries. Blog cards link titles to the first translation and display language-specific links for available translations.

Changes

Blog translation links

Layer / File(s) Summary
Translation data entries
_data/posts.yml
Five posts now define ordered English translation entries. The ministerial-change post also defines Sinhala and Tamil entries.
Translation link rendering
_includes/blog.html
Article cards use the first translation for the title link, remove the separate “Read more” link, and render available translation links.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • LDFLK/opendata#91: Both changes update blog post data and translation link rendering.

Suggested reviewers: zaeema-n

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The blog requirements are addressed, but no OpenGINXplore screenshot update appears in the provided file changes. Add or provide the OpenGINXplore application screenshot update required by issue #94.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes multilingual blog listings and the intended application screenshot update.
Out of Scope Changes check ✅ Passed The provided changes support multilingual blog listings and do not show unrelated code changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@zaeema-n zaeema-n left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 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 `@_includes/blog.html`:
- Around line 48-50: Update the title link in the blog post markup to resolve
its URL from post.translations by selecting the English translation entry, while
preserving relative_url processing and the existing post.title display. Do not
rely on the removed top-level post.url field.
🪄 Autofix

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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a69c562d-0f1d-4c46-ba84-61e2b32b4530

📥 Commits

Reviewing files that changed from the base of the PR and between 5b7cdd4 and a1fa592.

⛔ Files ignored due to path filters (1)
  • images/openginxplore.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • _data/posts.yml
  • _includes/blog.html

Comment thread _includes/blog.html
@ChanukaUOJ ChanukaUOJ changed the title fix: Update the application screenshots in the applications page feat: Enable the multilingual post listing support in the blog page & update the application screenshots Aug 12, 2026
@ChanukaUOJ ChanukaUOJ changed the title feat: Enable the multilingual post listing support in the blog page & update the application screenshots GH-94: feat: Enable the multilingual post listing support in the blog page & update the application screenshots Aug 12, 2026
@ChanukaUOJ
ChanukaUOJ requested a review from zaeema-n August 12, 2026 05:52

@zaeema-n zaeema-n left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ChanukaUOJ
ChanukaUOJ merged commit 69e3730 into LDFLK:main Aug 12, 2026
2 checks passed
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.

feat: Enable the multilingual post listing support in the blog page & update the application screenshots

2 participants