Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[master] Use toJson method from data.jsondata module in http client #2168

Closed
wants to merge 46 commits into from

Conversation

lnash94
Copy link
Member

@lnash94 lnash94 commented Oct 10, 2024

Purpose

Fix: ballerina-platform/ballerina-library#7012

Client data binding integration: ballerina-platform/ballerina-library#6747
Service data binding integration: ballerina-platform/ballerina-library#7011

Disable the below test because of data.json data module currently does not support the table types
and enable tests once this issue is done: ballerina-platform/ballerina-library#7020

  • testIntTableDatabinding()
  • testIntTableOrXmlArrayDatabinding()
  • testIntTableDatabindingByType()
  • testStringTableDatabinding()
  • testStringTableDatabindingByType()
  • testRecordTableDatabinding()
  • testRecordTableDatabindingByType()
  • testByteArrTableDatabinding()
  • testByteArrTableDatabindingByType()

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility
  • Checked the impact on OpenAPI generation

Dependent PR : ballerina-platform/module-ballerina-data.jsondata#35 once this is merged , test can be added with new data.jsondata timestamp

Closed PR : #2151

lnash94 and others added 30 commits September 18, 2024 17:38
…le-ballerina-http into integrate-data.json

# Conflicts:
#	ballerina-tests/http-client-tests/tests/sc_res_binding_tests.bal
@lnash94
Copy link
Member Author

lnash94 commented Oct 11, 2024

Copy link

sonarcloud bot commented Oct 29, 2024

@lnash94
Copy link
Member Author

lnash94 commented Oct 29, 2024

Will reopen once the test are done with fixing.

@lnash94 lnash94 closed this Oct 29, 2024
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.

Integrate toJson method from data.jsondata module in http client
1 participant