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

Change customer_id from integer to string in schemas #7

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jgillmanjr
Copy link

Description of change

Change the customer_id field in the conversion_paths and action_updates schemas to be a string, which falls in line with the current actions schema.

Manual QA steps

Risks

  • Existing integrations may fail if the target table is typed for an integer. However this may not be the case if the target database is able to automatically typecast the string to an integer.

Rollback steps

  • revert this branch

@cmerrick
Copy link
Contributor

Hi @jgillmanjr, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

@cmerrick
Copy link
Contributor

You did it @jgillmanjr!

Thank you for signing the Singer Contribution License Agreement.

kspeer825 and others added 2 commits January 27, 2021 09:59
* limit pip

* use v4 image, mark tickets for tests since no tests run

Co-authored-by: Kyle Speer <[email protected]>
@jgillmanjr
Copy link
Author

Is it possible to get a review on this?

@GeorgeA93
Copy link

Anything stopping this from being reviewed...?

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.

5 participants