Skip to content

feat(#191): add RaiseDisputeModal with form, submission, XHR progress…#352

Merged
amina69 merged 6 commits into
amina69:mainfrom
Clinton6801:fix/191-raise-dispute-modal
Jun 2, 2026
Merged

feat(#191): add RaiseDisputeModal with form, submission, XHR progress…#352
amina69 merged 6 commits into
amina69:mainfrom
Clinton6801:fix/191-raise-dispute-modal

Conversation

@Clinton6801
Copy link
Copy Markdown
Contributor

Closes #191

Changes

  • Added RaiseDisputeModal with reason textarea (30 char minimum), live counter, evidence upload, per-file XHR progress tracking, inline errors and success toast
  • Added useMutateRaiseDispute hook with XHR upload, per-file progress, JSON fallback, auth headers and query cache invalidation
  • Added RaiseDisputeTrigger button with role-based visibility (CUSTODY_ACTIVE + isAdopter/isShelter)
  • Updated MSW dispute handler to support both JSON and FormData POST requests
  • 24 unit tests covering all acceptance criteria

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 28, 2026

@Clinton6801 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Clinton6801
Copy link
Copy Markdown
Contributor Author

Hi, I've implemented issue #191 — RaiseDisputeModal with form validation, XHR evidence upload with per-file progress tracking, and full unit test coverage. Happy to make any changes if needed.

@amina69
Copy link
Copy Markdown
Owner

amina69 commented May 30, 2026

@Clinton6801 conflict, kindly fix it

@Clinton6801
Copy link
Copy Markdown
Contributor Author

Hello @amina69 , It's been two days now and update or what exactly is going on ?

@amina69
Copy link
Copy Markdown
Owner

amina69 commented Jun 2, 2026

@Clinton6801 , am sorry but you still have a little conflict in the package-lock..fix that, and if not ci pipeline error, ofcos i will merge your code

@Clinton6801
Copy link
Copy Markdown
Contributor Author

Clinton6801 commented Jun 2, 2026 via email

@amina69
Copy link
Copy Markdown
Owner

amina69 commented Jun 2, 2026

@Clinton6801 , check, ci error , build failed

Copy link
Copy Markdown
Contributor Author

@Clinton6801 Clinton6801 left a comment

Choose a reason for hiding this comment

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

Check this now, i'm not happy with the fact that you responded late. you had three days to make us fix this early enough

@amina69
Copy link
Copy Markdown
Owner

amina69 commented Jun 2, 2026

@Clinton6801 your CI pipeline is still failing

Copy link
Copy Markdown
Contributor Author

@Clinton6801 Clinton6801 left a comment

Choose a reason for hiding this comment

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

@amina69 , Please check this now and let me know, Thank you

@amina69 amina69 merged commit d27ce83 into amina69:main Jun 2, 2026
1 check passed
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.

[Frontend · Dispute system UI] Create RaiseDisputeModal — form and submission

2 participants