Skip to content

Dev#111

Merged
masudparvez2050 merged 3 commits intomainfrom
dev
Mar 15, 2026
Merged

Dev#111
masudparvez2050 merged 3 commits intomainfrom
dev

Conversation

@masudparvez2050
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 15, 2026 13:41
@masudparvez2050 masudparvez2050 merged commit 3d41162 into main Mar 15, 2026
2 checks passed
Copy link

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 improves error handling in the boat listing form and adds an engine deletion feature to the boat registration step 2 form.

Changes:

  • Enhanced error message extraction in BoatListingForm to check response.error in addition to response.message
  • Added ability to delete engines from the multi-engine form in Step2Form

Reviewed changes

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

File Description
src/app/seller-dashboard/my-listing/_components/BoatListingForm.tsx Improved error handling to extract messages from additional response fields (error alongside message)
src/app/register-boat/_components/RegisterBoat/_components/Step2Form/Step2Form.tsx Added engine delete button with trash icon and handleDeleteEngine logic

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

const newEngines = currentEngines.filter(
(_: unknown, index: number) => index !== indexToDelete,
);

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