Skip to content

Create Ride Redesign - #113

Merged
chrisgoggins merged 8 commits into
mainfrom
luciano/create-ride
Oct 1, 2026
Merged

chrisgoggins merged 8 commits into
mainfrom
luciano/create-ride

Conversation

@ludafieno

@ludafieno ludafieno commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Resolves #105

Checklist

  • Title is the same as the ticket title
  • Ticket is mentioned above
  • Changes fulfill the ticket's acceptance criteria
  • Relevant developers have been assigned
  • Relevant reviewers (EM, Designers, etc.) have been requested

@netlify

netlify Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for able-alliance failed. Why did it fail? →

Name Link
🔨 Latest commit 3939b95
🔍 Latest deploy log https://app.netlify.com/projects/able-alliance/deploys/6abd85ce3d64d10009e0ae7b

@ludafieno
ludafieno marked this pull request as ready for review September 29, 2026 05:56
@greptile-apps

greptile-apps Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Disclaimer: This comment was AI-generated and is not necessarily completely accurate. Please take code comments with a grain of salt.

RetriggerConfidence Score: 5/5

[Medium risk] Redesigns the ride creation form UI and logic.

This PR appears safe to merge based on the in-scope changes and resolved review threads.

Summary

The PR redesigns the ride-request form, limits date and time choices to service hours, updates map presentation, and adjusts the Route B browser test.

  • The previous Greptile findings are fixed, conceded, or resolved; no new in-scope finding was accepted.
  • Review score: 100/100.

Reviews (4) · Last reviewed commit: "Merge branch 'main' into luciano/create-..."

Comment thread src/app/rides/new/page.tsx
Comment thread src/app/rides/new/page.tsx Outdated
Comment thread src/app/rides/new/page.tsx Outdated
Comment thread src/app/rides/new/styles.module.css Outdated
Comment thread src/app/rides/new/styles.module.css Outdated
Comment thread src/app/rides/new/page.tsx
@ludafieno

Copy link
Copy Markdown
Collaborator Author

@greptileai

Comment thread src/app/rides/new/styles.module.css Outdated
Comment thread src/app/rides/new/styles.module.css
@ludafieno

Copy link
Copy Markdown
Collaborator Author

@greptileai

Comment thread src/app/rides/new/styles.module.css Outdated
@ludafieno

Copy link
Copy Markdown
Collaborator Author

@greptileai

@chrisgoggins chrisgoggins left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@chrisgoggins
chrisgoggins merged commit 677cc77 into main Oct 1, 2026
2 of 6 checks passed
@chrisgoggins
chrisgoggins deleted the luciano/create-ride branch October 1, 2026 01:21
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.

Create Ride Redesign

2 participants