Skip to content

fix: rename proof upload field from notes to recipientSignatureName#196

Open
ahmedabbas52233-a11y wants to merge 2 commits into
Navin-xmr:mainfrom
ahmedabbas52233-a11y:feature/issue-179-proof-upload-field
Open

fix: rename proof upload field from notes to recipientSignatureName#196
ahmedabbas52233-a11y wants to merge 2 commits into
Navin-xmr:mainfrom
ahmedabbas52233-a11y:feature/issue-179-proof-upload-field

Conversation

@ahmedabbas52233-a11y
Copy link
Copy Markdown

Summary

Closes #179

Updates the proof upload API method to use the correct field name expected by the backend.

Changes

  • Renamed uploadProof parameter from notes?: string to recipientSignatureName: string.
  • Changed FormData field key from "notes" to "recipientSignatureName"
  • Made parameter required (no longer optional) per backend contract

Testing

  • pnpm run lint passes
  • pnpm run build passes
  • pnpm run dev starts without errors

- Update uploadProof parameter from optional notes to required recipientSignatureName
- Change FormData field key from notes to recipientSignatureName
- Align API client with backend contract for delivery proof uploads

Closes Navin-xmr#179
- Update uploadProof parameter from optional notes to required recipientSignatureName
- Change FormData field key from notes to recipientSignatureName
- Add *.tsbuildinfo to .gitignore to prevent build cache from being committed
- Align API client with backend contract for delivery proof uploads

Closes Navin-xmr#179
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.

[Bug Fix] Update proof upload to send recipientSignatureName instead of notes

1 participant