Skip to content

update docs#283

Merged
patrickcook28 merged 1 commit intomainfrom
ERP-5281/update-erp-docs-to-include-custom-fields
Mar 3, 2025
Merged

update docs#283
patrickcook28 merged 1 commit intomainfrom
ERP-5281/update-erp-docs-to-include-custom-fields

Conversation

@patrickcook28
Copy link
Contributor

This PR adds custom fields to the ERP developer guide

Copilot AI review requested due to automatic review settings March 3, 2025 17:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This PR updates the ERP developer guide by adding custom fields examples to the event payloads and refining existing payload examples.

  • Introduces custom_fields objects with structured examples in various event payloads
  • Updates the request_name in an event payload example and changes erp_custom_fields from null to an empty object

Reviewed Changes

File Description
erp_integration/erp_events_dictionary.md Added custom_fields examples and adjusted existing payload representations

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (3)

erp_integration/erp_events_dictionary.md:379

  • [nitpick] Verify that the use of 'custom_field_1' and 'custom_field_2' is consistent with the intended design, and update the documentation to clearly explain the difference, if any, between these fields.
"custom_fields": {

erp_integration/erp_events_dictionary.md:1943

  • Ensure that the updated 'request_name' is supported in the backend integration and that related documentation is updated accordingly.
"request_name": "create_prime_contract_change_order",

erp_integration/erp_events_dictionary.md:1150

  • Changing 'erp_custom_fields' from null to an empty object could affect downstream processing; please confirm that the integration accepts the new format.
"erp_custom_fields": {},

@patrickcook28 patrickcook28 merged commit fc11f75 into main Mar 3, 2025
4 checks passed
@patrickcook28 patrickcook28 deleted the ERP-5281/update-erp-docs-to-include-custom-fields branch March 3, 2025 19:57
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.

4 participants