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 #2203

Closed
wants to merge 47 commits into from

Conversation

lnash94
Copy link
Member

@lnash94 lnash94 commented Nov 1, 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 lnash94 force-pushed the integrate-data.json branch 3 times, most recently from 81682a2 to f63ed48 Compare November 1, 2024 06:33
Copy link

sonarcloud bot commented Nov 1, 2024

@lnash94 lnash94 closed this Nov 3, 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