-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
✨Source Auth0: Migrate Python CDK to Low-code CDK #28972
Conversation
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
PS: All tests are passing, Please ping via slack for getting credentials. |
@btkcodedev let's wait for #29001 to get merged. |
@btkcodedev can you fix the conflicts and implement the new 3 streams? Thanks! |
airbyte-integrations/connectors/source-auth0/source_auth0/manifest.yaml
Outdated
Show resolved
Hide resolved
…into auth0lowcode
airbyte-integrations/connectors/source-auth0/source_auth0/manifest.yaml
Outdated
Show resolved
Hide resolved
@sajarin @marcosmarxm Test result - All passed state :) |
@sajarin setup.py had conflicts, Cleared 'em all and updated expected records, |
airbyte-integrations/connectors/source-auth0/source_auth0/manifest.yaml
Outdated
Show resolved
Hide resolved
@sajarin Pagination implemented |
Thanks @btkcodedev, currently running tests in the PR above, the tests seem to have timed out before so investigating that at the moment. |
@btkcodedev tested using the new credentials you shared through slack, running into failing incremental tests: namely test_two_sequential_reads and test_read_sequential_slices. Both seem to fail when comparing the state and records cursors: |
…into auth0lowcode
…into auth0lowcode
Thanks for the contribution @btkcodedev! |
Thank you @sajarin for your continuous efforts 🔥💫❤️ |
Co-authored-by: Sajarin <[email protected]>
What
Migrating Source Auth0 to Low-Code CDK
Closes #28928
How
Developed using (Configuration Based Source) low-code CDK
Recommended reading order
spec.yaml
manifest.yaml
schemas/*
Tests
Integration & Acceptance
Full Test Results🚨 User Impact 🚨
Migrate Connector: Source Auth0
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
docs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampledocs/integrations/README.md
airbyte-integrations/builds.md