-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
🎉 New Source: Merge [Low code CDK] #25342
Conversation
PS for the reviewers: Thus the implemented endpoints will give basic information about the merge and its integrated account. |
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.
There are others endpoints. Are you planning to add them too? Left some comments of missing features.
airbyte-integrations/connectors/source-merge/source_merge/manifest.yaml
Outdated
Show resolved
Hide resolved
Here are the credentials: No worries, Completely made up of temp email creds. |
retriever: | ||
$ref: "#/definitions/retriever" | ||
|
||
account_details_stream: |
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.
Why you're not adding the /accounts
endpoint?
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.
The /accounts is for Accounting Base, the issue was based only for ATS URL
See https://docs.merge.dev/ats/overview/
All the endpoints inside Merge ATS is added :)
@marcosmarxm Bump here :) |
/test connector=connectors/source-merge
Build PassedTest summary info:
|
/publish connector=connectors/source-merge
if you have connectors that successfully published but failed definition generation, follow step 4 here |
@marcosmarxm Please add hackathon-developer-week label |
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 @btkcodedev I'm having problems publishing the connector. Request help from our connector ops team. All your tasks are done!
Thank you for your continued support & efforts @marcosmarxm |
/publish connector=connectors/source-merge
if you have connectors that successfully published but failed definition generation, follow step 4 here |
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 @btkcodedev
* Initial commit * add pagination * add streams activites, applications, attachments, candidates, departments * add schemas and stream manifest for eeocs, interviews, offers, offices, jobs, job-interview-stages * Update spec.yaml * fix test * add uuid for merge connector * update job interview stages * remove seed files unused * auto-bump connector version --------- Co-authored-by: marcosmarxm <marcosmarxm@gmail.com> Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com> Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
What
Developing a new connector for the source: Merge
Resolves:
Merge
Part of hackathon week
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 🚨
New Connector: Source Merge
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