-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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: Wrike
#15638
🎉 New Source: Wrike
#15638
Conversation
Thanks for the contribution @tfoldi please let us know when it's ready to review. |
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.
Hi @tfoldi, thank you for contributing. Your code looks good so far. I left some comments on things you can improve for readability.
airbyte-integrations/connectors/source-wrike/source_wrike/schemas/comments.json
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-wrike/source_wrike/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-wrike/source_wrike/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-wrike/source_wrike/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-wrike/unit_tests/test_streams.py
Outdated
Show resolved
Hide resolved
sure, just implemented the changes @tuanchris |
@tfoldi can you post a screenshot of passing acceptance tests? |
/publish connector=connectors/source-wrike
if you have connectors that successfully published but failed definition generation, follow step 4 here |
/publish connector=connectors/source-wrike run-tests=false
if you have connectors that successfully published but failed definition generation, follow step 4 here |
Thanks for the connector @tfoldi and thanks for the review @tuanchris! |
* Connector skeleton * Implement the first two endpoints with incomplete schema * Add more fields to download from tasks * Add more fields to download from tasks * Add new entities plus refector a few stuff * Complete all schema jsons * Add slice management for comments * complete schema definition for folders * remove incremental tests * Removing as no incremental support yet * Format as pep8 * sample config files * add newer version of the catalog * remove unused import * use wrike cataglog * remove additional properties * remove incremental tests * reformat everything with Prettifier * Add developer notes * Add source-wrike badge * Add Wrike source as alpha * Document Wrike connector usage * Add title fields for input parameters * fix typo * handle empty start date string * add unit tests for wrike source * remove unnecessary comments * duplicated file * document how to get permanent tokens * Add link to API documentation * Apply gradlew format * Add workflow endpoint * Change youtube back to Beta * Remove unnecesary TODO comment * Add null constraints * Code cleanup in check_connection and streams * Improved null constraints in schema * Convert arrays with one element to string * chore: update source_definitions.yaml * chore: generate/update source_specs.yaml Co-authored-by: sajarin <sajarindider@gmail.com>
* Connector skeleton * Implement the first two endpoints with incomplete schema * Add more fields to download from tasks * Add more fields to download from tasks * Add new entities plus refector a few stuff * Complete all schema jsons * Add slice management for comments * complete schema definition for folders * remove incremental tests * Removing as no incremental support yet * Format as pep8 * sample config files * add newer version of the catalog * remove unused import * use wrike cataglog * remove additional properties * remove incremental tests * reformat everything with Prettifier * Add developer notes * Add source-wrike badge * Add Wrike source as alpha * Document Wrike connector usage * Add title fields for input parameters * fix typo * handle empty start date string * add unit tests for wrike source * remove unnecessary comments * duplicated file * document how to get permanent tokens * Add link to API documentation * Apply gradlew format * Add workflow endpoint * Change youtube back to Beta * Remove unnecesary TODO comment * Add null constraints * Code cleanup in check_connection and streams * Improved null constraints in schema * Convert arrays with one element to string * chore: update source_definitions.yaml * chore: generate/update source_specs.yaml Co-authored-by: sajarin <sajarindider@gmail.com>
What
This a new source connector for Wrike, which is a work management software used by small and large companies. The source connector works fairly well, but I need some unit tests and feedback from you guys. (So it is WIP)
How
Recommended reading order
It is a plan custom source, the most boring one ever.
🚨 User Impact 🚨
No breaking changes.
Pre-merge Checklist
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampledocs/integrations/README.md
airbyte-integrations/builds.md
Airbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described hereTests
Unit
Integration
Acceptance