🐛 Source Iterable: Fixed normalization issue

[yaroslav-dudar]

What

Fixed schema in streams: campaigns, lists, templates, metadata #5867

How

Describe the solution

Recommended reading order

1. x.java
2. y.python

Pre-merge Checklist

Expand the relevant checklist and delete the others.

New Connector

Community member or Airbyter

• Community member? Grant edit access to maintainers (instructions)
• Secrets in the connector's spec are annotated with airbyte_secret
• Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
• Code reviews completed
• Documentation updated
  • Connector's README.md
  • Connector's bootstrap.md. See description and examples
  • docs/SUMMARY.md
  • docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
  • docs/integrations/README.md
  • airbyte-integrations/builds.md
• PR name follows PR naming conventions
• Connector added to connector index like described here

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

• Create a non-forked branch based on this PR and test the below items on it
• Build is successful
• Credentials added to Github CI. Instructions.
• /test connector=connectors/<name> command is passing.
• New Connector version released on Dockerhub by running the /publish command described here

Updating a connector

Community member or Airbyter

• Grant edit access to maintainers (instructions)
• Secrets in the connector's spec are annotated with airbyte_secret
• Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
• Code reviews completed
• Documentation updated
  • Connector's README.md
  • Connector's bootstrap.md. See description and examples
  • Changelog updated in docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
• PR name follows PR naming conventions
• Connector version bumped like described here

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

• Create a non-forked branch based on this PR and test the below items on it
• Build is successful
• Credentials added to Github CI. Instructions.
• /test connector=connectors/<name> command is passing.
• New Connector version released on Dockerhub by running the /publish command described here

Connector Generator

• Issue acceptance criteria met
• PR name follows PR naming conventions
• If adding a new generator, add it to the list of scaffold modules being tested
• The generator test modules (all connectors with -scaffold in their name) have been updated with the latest scaffold by running ./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates then checking in your changes
• Documentation which references the generator is updated as needed.

[yaroslav-dudar]

maybe we need to wait for this PR #5915 . There are an acceptance test added , etc

[midavadim]

Please ensure that SAT are passed

[yaroslav-dudar]

@davinchia we have parallel PR here. I'm not sure which one has higher priority #6242 (comment)

[marcosmarxm]

@davinchia we have parallel PR here. I'm not sure which one has higher priority #6242 (comment)

can we merge this first? there is a user having problem because of this issue. check slack convo

[davinchia]

@marcosmarxm is integer the right datatype to have?

@yaroslav-dudar please make sure the tests passes before merging in.

[yaroslav-dudar]

@davinchia there no acceptance tests for this connector right now. guys added tests in this PR #5915, but I don't know if it's close to finalizing

[davinchia]

@yaroslav-dudar got it. Let me check with Marcos if this is right.

[marcosmarxm]

@marcosmarxm is integer the right datatype to have?

yes, https://api.iterable.com/api/docs

[yaroslav-dudar]

/publish connector=connectors/source-iterable

🕑 connectors/source-iterable https://github.com/airbytehq/airbyte/actions/runs/1270238634
❌ connectors/source-iterable https://github.com/airbytehq/airbyte/actions/runs/1270238634

[yaroslav-dudar]

/publish connector=connectors/source-iterable

🕑 connectors/source-iterable https://github.com/airbytehq/airbyte/actions/runs/1270264975
✅ connectors/source-iterable https://github.com/airbytehq/airbyte/actions/runs/1270264975