Skip to content

Conversation

@JasonWarrenUK
Copy link
Collaborator

@JasonWarrenUK JasonWarrenUK commented Jun 9, 2025

Overview

Renders the correct questions/actions in List view based on the selected dashboard tile.


TL;DR

Run the new dev script before starting the app:

npm run db:reset

Gallery

CleanShot.2025-06-09.Monday.at.3.09.08.Arc.mp4

Changes

'tis fruity

Database

Added a preview string prop to the questions table, then tweaked the seed files to use it.

List

It's great. Queries question.category to populate the list then renders an element using question.preview.

QuestionCard

Incorporates some of @Jaz-spec's work on this component

Misc

  • Tweaked the dev outline style to reduce the chance of inducing some form of seizure when looking at them.

@JasonWarrenUK JasonWarrenUK requested a review from a team June 9, 2025 14:22
Copy link
Collaborator

@AlexVOiceover AlexVOiceover left a comment

Choose a reason for hiding this comment

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

Just a suggestion about consolidation Button components in a single one.
No need to be done on this PR (Generic Button will likely evolve to acomodate all buttons)

@JasonWarrenUK JasonWarrenUK requested review from a team and AlexVOiceover June 10, 2025 11:46
Copy link
Collaborator

@Jaz-spec Jaz-spec left a comment

Choose a reason for hiding this comment

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

Yep, looks good to me!

Copy link
Collaborator

@Jaz-spec Jaz-spec left a comment

Choose a reason for hiding this comment

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

Yep, looks good to me!

@JasonWarrenUK JasonWarrenUK merged commit 3289211 into main Jun 10, 2025
1 check passed
@JasonWarrenUK JasonWarrenUK deleted the feat/list-logic branch June 10, 2025 13:31
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.

4 participants