Skip to content

refactor articles, add article index, opengraph metadata#17

Merged
ccbrown merged 2 commits intomainfrom
articles-refactor
Apr 19, 2025
Merged

refactor articles, add article index, opengraph metadata#17
ccbrown merged 2 commits intomainfrom
articles-refactor

Conversation

@ccbrown
Copy link
Owner

@ccbrown ccbrown commented Apr 19, 2025

What It Does

Refactors the articles route to add an index, add OpenGraph metadata, and prepare for more articles.

Copilot AI review requested due to automatic review settings April 19, 2025 04:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the articles route to add an index page, include OpenGraph metadata, and prepare for additional articles.

  • Implements an index page that sorts and displays articles.
  • Introduces a centralized articles definition source.
  • Updates dynamic routing in the [slug] page to generate static parameters with OpenGraph metadata.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

File Description
frontend/src/app/(public-area)/articles/page.tsx Creates the articles index page with sorted articles listing.
frontend/src/app/(public-area)/articles/index.tsx Exports a centralized articles object with article definitions.
frontend/src/app/(public-area)/articles/capital-one-data-breach.tsx Defines updated article content for the Capital One breach article.
frontend/src/app/(public-area)/articles/[slug]/page.tsx Implements dynamic routing with static params and OpenGraph metadata.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ccbrown ccbrown merged commit d90e4be into main Apr 19, 2025
3 checks passed
@ccbrown ccbrown deleted the articles-refactor branch April 19, 2025 05:05
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.

1 participant