Skip to content

Improve recent retros view#25

Merged
ellgreen merged 1 commit intomainfrom
recent-retros-improvements
May 11, 2025
Merged

Improve recent retros view#25
ellgreen merged 1 commit intomainfrom
recent-retros-improvements

Conversation

@ellgreen
Copy link
Copy Markdown
Owner

@ellgreen ellgreen commented May 11, 2025

For #18

@ellgreen ellgreen requested a review from Copilot May 11, 2025 19:58
Copy link
Copy Markdown

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 aims to enhance the recent retros view by updating both frontend and backend components. Key changes include:

  • Adding new fields (created_at, note_count, task_count, task_completed_count) to the Retro model and corresponding TypeScript interface.
  • Adjusting UI components and layout improvements for displaying retros.
  • Updating backend SQL queries and models to aggregate new retro metadata.

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
ui/src/types.ts Added new fields to the Retro interface.
ui/src/routes/_auth.index.tsx Modified grid layout for the retros view.
ui/src/components/retro/list.tsx Enhanced retro item display with counts, dates, and updated styling.
ui/src/components/nav.tsx Adjusted Dropdown menu alignment.
cmd/thoughts/resources/retro.go Mapped new retro fields from the model to the API response.
cmd/thoughts/model/retro.go Included new columns in the Retro model.
cmd/thoughts/dal/retro.go Updated SQL query to join and group new retro fields and changed ordering to created_at desc.
Comments suppressed due to low confidence (1)

cmd/thoughts/dal/retro.go:28

  • Verify that changing the ordering from 'updated_at' to 'created_at' is intentional to correctly reflect the recent retros view.
group by retros.id order by retros.created_at desc limit 25

Comment thread ui/src/components/retro/list.tsx
@ellgreen ellgreen merged commit 3b66988 into main May 11, 2025
@ellgreen ellgreen deleted the recent-retros-improvements branch May 11, 2025 19:59
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