Skip to content

CustomCommunication class and streamline SMS logs handling by relying on server-side defaults for columns and rows#12

Merged
vladiacob merged 7 commits intohackajob:hackajob-developmentfrom
nidh-eesh:nidheesh-milestone-2
Feb 13, 2026
Merged

CustomCommunication class and streamline SMS logs handling by relying on server-side defaults for columns and rows#12
vladiacob merged 7 commits intohackajob:hackajob-developmentfrom
nidh-eesh:nidheesh-milestone-2

Conversation

@nidh-eesh
Copy link

This pull request introduces a backend override for the Communication doctype to provide custom default columns and rows for SMS log listings, and removes redundant frontend logic that previously enforced these defaults. Now, the frontend relies on the backend to supply the correct structure, simplifying the Vue component and centralizing configuration.

Backend enhancements:

  • Added a CustomCommunication class in crm/overrides/communication.py that overrides the default columns and rows for the Communication doctype, ensuring consistent structure for SMS logs.
  • Registered the new CustomCommunication override in the override_doctype_class mapping in crm/hooks.py.

Frontend simplification:

  • Removed the Vue watch and onMounted logic in frontend/src/pages/SmsLogs.vue that previously set default columns and rows for SMS logs, as this is now handled by the backend. [1] [2]

@vladiacob vladiacob merged commit 3935ac8 into hackajob:hackajob-development Feb 13, 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.

2 participants