Skip to content

refactor: split jmap.py into services#401

Merged
s-aga-r merged 1 commit into
frappe:developfrom
s-aga-r:refactor-jmap
Mar 19, 2026
Merged

refactor: split jmap.py into services#401
s-aga-r merged 1 commit into
frappe:developfrom
s-aga-r:refactor-jmap

Conversation

@s-aga-r
Copy link
Copy Markdown
Collaborator

@s-aga-r s-aga-r commented Mar 18, 2026

Split JMAPClient into modular service classes.

Next:

  • Use data models for request/response objects.
  • Deduplicate JMAP method response handling.
  • Add configurable logging for JMAP API calls.

@s-aga-r s-aga-r force-pushed the refactor-jmap branch 4 times, most recently from ee2cdc1 to e0c72cf Compare March 18, 2026 17:57
@s-aga-r s-aga-r marked this pull request as ready for review March 19, 2026 04:47
@s-aga-r s-aga-r requested a review from krantheman as a code owner March 19, 2026 04:47
@s-aga-r s-aga-r merged commit fda4088 into frappe:develop Mar 19, 2026
3 checks passed
@s-aga-r s-aga-r deleted the refactor-jmap branch March 31, 2026 11:36
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