Skip to content

add new page to host all unlisted blog links#1811

Merged
tgberkeley merged 2 commits intomainfrom
new-index-page
Apr 7, 2026
Merged

add new page to host all unlisted blog links#1811
tgberkeley merged 2 commits intomainfrom
new-index-page

Conversation

@tgberkeley
Copy link
Copy Markdown
Collaborator

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 3, 2026

Greptile Summary

This PR adds a new /page-index route (pcweb/pages/page_index.py) that renders a simple list of blog post links, and re-exports it from pcweb/pages/__init__.py. Despite the PR title "ruff fix", the change is purely additive — no linting fixes are visible in the diff.

The implementation is clean and follows existing project conventions (uses rx.el.* elements, @webpage decorator, Tailwind classes). The only concern is that BLOG_POSTS is a hardcoded list of only 5 entries, which will silently go stale as new posts are added or existing URLs change.

Confidence Score: 5/5

Safe to merge; only concern is the hardcoded blog post list which is a minor maintenance issue.

All findings are P2 (style/maintenance). No logic bugs, security issues, or correctness problems. The new page follows existing project conventions correctly.

pcweb/pages/page_index.py — the hardcoded BLOG_POSTS list may require ongoing manual maintenance.

Important Files Changed

Filename Overview
pcweb/pages/page_index.py New page at /page-index with a hardcoded list of 5 blog posts; clean implementation following project conventions.
pcweb/pages/init.py Adds re-export of the new page_index page, consistent with the existing import style in this file.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["GET /page-index"] --> B["page_index() component"]
    B --> C["rx.el.div (max-w-2xl)"]
    C --> D["rx.el.h1 — 'Page Index'"]
    C --> E["rx.el.ul"]
    E --> F["link_item(title, path) × 5"]
    F --> G["rx.el.li → rx.el.a (href=path)"]
    subgraph BLOG_POSTS [Hardcoded BLOG_POSTS list]
        H["Build Python Admin Panels…"]
        I["Reflex vs Plotly Dash"]
        J["Top Python Web Frameworks 2026"]
        K["Designing a Pure Python Web Framework"]
        L["Reflex vs Streamlit"]
    end
    F -.->|iterates| BLOG_POSTS
Loading

Reviews (1): Last reviewed commit: "ruff fix" | Re-trigger Greptile

@tgberkeley tgberkeley changed the title ruff fix add new page to host all unlisted blog links Apr 3, 2026
@tgberkeley tgberkeley merged commit f42301b into main Apr 7, 2026
9 of 10 checks passed
@tgberkeley tgberkeley deleted the new-index-page branch April 7, 2026 18:06
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.

2 participants