Skip to content

List Item Components #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 25 commits into from
Closed

List Item Components #16

wants to merge 25 commits into from

Conversation

@JasonWarrenUK
Copy link
Collaborator

@JasonWarrenUK JasonWarrenUK commented Jun 9, 2025

Overview

Creates a separate LiftItem component with placeholders for status indicator etc

Jaz-spec and others added 22 commits June 6, 2025 14:16
@Jaz-spec
basic component with dummy data in place
75d61d7
@Jaz-spec
skeleton for displaying question data in place
2756e51
@Jaz-spec
commit to merge updated types
ef938f4
@Jaz-spec
commit
6361445
@Jaz-spec
data is being displayed from the db
e724a65
@Jaz-spec
added basic question saving for version 1 of a question
0b098b6
@Jaz-spec
integrated action types
c8f2be9
@Jaz-spec
ability to retrieve previous answers
fe51e2b
@Jaz-spec
responses marked as either answered or skipped
b558d9b
@Jaz-spec
retrieved and displayed latest action in question card component
33cc0be
@Jaz-spec
public private toggle implemented
3ec4c57
@Jaz-spec
fixed some toggle mis-haps
f3fb76e
@JasonWarrenUK
Merge branch 'question-component' of https://github.com/foundersandco…
87e1624 
…ders/LIFT02 into feat/list-logic
@JasonWarrenUK
style: 💄 rationalise imports from different branches
31bb8f2
@JasonWarrenUK
style: 💄 remove dashed horrors
3621fe6
@JasonWarrenUK
feat: 🏷️ create list category type
8b235b5
@JasonWarrenUK
feat: 🗃️ add preview field to question table
d36ba26
@JasonWarrenUK
feat: 💄 render the correct questions/actions in list view
3b7e1e0
@JasonWarrenUK
Merge branch 'header_component' of https://github.com/foundersandcode…
8d535b9 
…rs/LIFT02 into feat/list-logic
@JasonWarrenUK
refactor(project): 🚚 reorganise components
86e3a4b
@JasonWarrenUK
feat: 💄 create separate list item component
59dc132
@JasonWarrenUK
build: 👷 make pr template not poo
82d3cb2
@JasonWarrenUK JasonWarrenUK requested a review from a team June 9, 2025 15:25
AlexVOiceover
AlexVOiceover reviewed Jun 10, 2025
View reviewed changes
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.

All good, will approve when #15 is resolved

@JasonWarrenUK JasonWarrenUK changed the title Feat/list-item-structure List Item Components Jun 10, 2025
@JasonWarrenUK JasonWarrenUK requested a review from a team June 10, 2025 11:46
@JasonWarrenUK JasonWarrenUK changed the base branch from feat/list-logic to main June 10, 2025 11:49
AlexVOiceover
AlexVOiceover previously approved these changes Jun 10, 2025
View reviewed changes
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.

ok

@JasonWarrenUK JasonWarrenUK dismissed AlexVOiceover’s stale review June 10, 2025 13:40

The merge-base changed after approval.

@JasonWarrenUK JasonWarrenUK requested a review from a team June 10, 2025 13:42
AlexVOiceover
AlexVOiceover approved these changes Jun 10, 2025
View reviewed changes
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.

ok

AlexVOiceover
AlexVOiceover previously approved these changes Jun 10, 2025
View reviewed changes
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.

ok

AlexVOiceover
AlexVOiceover previously approved these changes Jun 10, 2025
View reviewed changes
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.

ok

@AlexVOiceover AlexVOiceover self-requested a review June 10, 2025 13:45
@JasonWarrenUK JasonWarrenUK dismissed AlexVOiceover’s stale review June 10, 2025 13:45

The merge-base changed after approval.

AlexVOiceover
AlexVOiceover approved these changes Jun 10, 2025
View reviewed changes
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.

ok

@Jaz-spec Jaz-spec deleted the feat/list-item-structure branch June 10, 2025 13:48
AlexVOiceover added a commit that referenced this pull request Nov 5, 2025
@AlexVOiceover
fix(ui): convert all titles and buttons to sentence case and update a…
950e960 
…uth config

- Convert all modal titles, button labels, and form fields from Title Case to sentence case (Issue #16)
- Update supabase config.toml to include both localhost and 127.0.0.1 auth callback URLs
- Add troubleshooting note in README for magic link authentication issues
- Update Supabase CLI to v2.54.1 to resolve magic link callback generation
- Update TESTING_ISSUES.md with completed Issue #16 details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexVOiceover AlexVOiceover AlexVOiceover approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@JasonWarrenUK @AlexVOiceover @Jaz-spec