-
Notifications
You must be signed in to change notification settings - Fork 4k
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 Orbit : Migrated to Low code #30976
✨ Source Orbit : Migrated to Low code #30976
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
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,
|
Thanks for the contribution @avirajsingh7. I added it to the team backlog and we're planning to start next week the review process. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made a quick review asking to update a few files this will help us during the code review. Sorry the long delay to get into your contribution. Probably this week or next one the team will start the process. Are you able to run the tests in your local setup? It can speed up a lot the process if you share the integration tests results
airbyte-integrations/connectors/source-orbit/source_orbit/manifest.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @avirajsingh7
What
Migrating Source Orbit to Low-Code CDK
Closes #30892
How
Developed using (Configuration Based Source) low-code CDK
Recommended reading order
🚨 User Impact 🚨
Test
Acceptance Test
Migrate Connector : Source Orbit