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

🐛 Fix: change all integers to number on Source Recurly catalog #3769

Merged
merged 1 commit into from Jun 1, 2021

Conversation

marcosmarxm
Copy link
Member

What

From user:

I've notice that there is a new version of the Recurly Source connector (0.2.2) I've tried with that version and It seems that the invoice table worked correctly but as I've mentioned before that wasn't the only table throwing error, plans , subscriptions and transactions are the tables, I attach the full log from the server

How

Change all integers to numbers on catalog.json.

Pre-merge Checklist

  • Run integration tests
  • Publish Docker images

Recommended reading order

1 .json files
2. the rest

@marcosmarxm
Copy link
Member Author

marcosmarxm commented May 31, 2021

/test connector=source-recurly

🕑 source-recurly https://github.com/airbytehq/airbyte/actions/runs/893995266
✅ source-recurly https://github.com/airbytehq/airbyte/actions/runs/893995266

@marcosmarxm
Copy link
Member Author

marcosmarxm commented May 31, 2021

/publish connector=connectors/source-recurly

🕑 connectors/source-recurly https://github.com/airbytehq/airbyte/actions/runs/894010872
✅ connectors/source-recurly https://github.com/airbytehq/airbyte/actions/runs/894010872

@marcosmarxm marcosmarxm merged commit 3129a17 into master Jun 1, 2021
@marcosmarxm marcosmarxm deleted the marcosmarxm/recurly-change-catalog-int-number branch June 1, 2021 18:09
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.

None yet

2 participants