Skip to content

fix: Handle bytes and use Flask get_json for request parsing#4

Merged
sammish merged 1 commit into
mainfrom
develop
Dec 17, 2025
Merged

fix: Handle bytes and use Flask get_json for request parsing#4
sammish merged 1 commit into
mainfrom
develop

Conversation

@sammish
Copy link
Copy Markdown
Contributor

@sammish sammish commented Dec 17, 2025

Frappe doesn't always pass nested JSON arrays through kwargs. Now explicitly reads frappe.request.data to get the full JSON body including the items array.

- Use Flask's get_json() method which handles content-type and encoding
- Fallback to manual parsing with proper bytes decoding
- Enhanced debug logging to troubleshoot request data issues

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@sammish sammish merged commit cf1e6a0 into main Dec 17, 2025
1 of 4 checks 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.

1 participant