Skip to content

feat(2403): wire up learn page#2446

Open
ycanales wants to merge 1 commit into
developfrom
cy/2403-learn-page
Open

feat(2403): wire up learn page#2446
ycanales wants to merge 1 commit into
developfrom
cy/2403-learn-page

Conversation

@ycanales
Copy link
Copy Markdown
Collaborator

@ycanales ycanales commented May 16, 2026

Issue: #2403

Summary & Context

Changes

  • Uses get_latest_post_cards
  • Adds Category.short_description: field, migration and Waftail snippet.
  • Randomised category-card builder in LearnPageView
  • Link it to the "Learn" in navbar.

‼️ Risks & Considerations ‼️

  • Implement Entry-Library relationship for library tags?
image image image

Screenshots

learn

Self-review Checklist

  • Tag at least one team member from each team to review this PR
  • Link this PR to the related GitHub Project ticket

Frontend

  • UI implementation matches Figma design
  • Tested in light and dark mode
  • Responsive / mobile verified
  • Accessibility checked (keyboard navigation, etc.)
  • Ensure design tokens are used for colors, spacing, typography, etc. – No hardcoded values
  • Test without JavaScript (if applicable)
  • No console errors or warnings

@ycanales ycanales requested review from herzog0 and jlchilders11 May 18, 2026 17:20
@ycanales ycanales linked an issue May 18, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

@herzog0 herzog0 left a comment

Choose a reason for hiding this comment

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

Hey Cristian, all good from my end, just a couple of comments that you should be aware of!

Comment thread core/views.py
kwargs={
"version_slug": LATEST_RELEASE_URL_PATH_STR,
"library_view_str": "categorized",
"category_slug": category.slug,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't think this is doing anything currently. We should probably create a follow-up ticket to wire this up correctly to what @julhoang did in her pr #2420

<div class="item-a">
{% with data=post_cards_data %}
{% include 'v3/includes/_post_cards_v3.html' with heading=data.heading posts=data.posts view_all_url=data.view_all_url view_all_label=data.view_all_label theme='teal' variant="content-card" %}
{% include 'v3/includes/_post_card.html' with heading=data.heading items=data.posts variant="card" theme="teal" layout="vertical" primary_cta_label=data.view_all_label primary_cta_url=data.view_all_url %}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just a heads up that this will conflict with @jlchilders11's pr #2414.
In his PR he also fixes the designs of the boost community card, so QA must be aware that you're tackling this in your PR, if it moves to QA before Jeremy's one gets merged.

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.

Webpage Integration: Learn

2 participants