Skip to content

Conversation

@deo002
Copy link
Collaborator

@deo002 deo002 commented Aug 4, 2025

Changes

  • External Ref configuration added to obligation model to save internal data fields.

Submitter Checklist

  • Includes tests (if there is a feature changed/added)
  • Includes docs ( if changes are user facing)
  • I have tested my changes locally.

@deo002 deo002 requested review from GMishx and Kaushl2208 and removed request for Kaushl2208 August 7, 2025 05:23
Copy link
Member

@Kaushl2208 Kaushl2208 left a comment

Choose a reason for hiding this comment

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

Just a small change recommendation. Will test it

"comment": {
"type": "string"
},
"externalRef": {
Copy link
Member

Choose a reason for hiding this comment

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

Can we bring more consistency in naming? I see there is externalRef in json and external_ref in Go Structs?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes Kaushl, This is the schema for Go struct in swagger. In json, it's external_ref.

@Kaushl2208
Copy link
Member

@deo002 , While creating obligations getting this:
image

@deo002 deo002 force-pushed the feat/obligationExternalRef branch 2 times, most recently from 9e01c84 to 2fb3624 Compare August 21, 2025 10:54
@deo002 deo002 force-pushed the feat/obligationExternalRef branch from 2fb3624 to 266c938 Compare August 21, 2025 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants