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

🎉 Source ZohoCRM - enable in cloud #23906

Merged
merged 25 commits into from
Mar 19, 2023

Conversation

midavadim
Copy link
Contributor

Enable in cloud

What

fixed authentication to support latest CDK,
fixed SAT,
updated expected records

How

Describe the solution

Recommended reading order

  1. x.java
  2. y.python

🚨 User Impact 🚨

Are there any breaking changes? What is the end result perceived by the user? If yes, please merge this PR with the 🚨🚨 emoji so changelog authors can further highlight this if needed.

Pre-merge Checklist

Expand the relevant checklist and delete the others.

New Connector

Community member or Airbyter

  • Community member? Grant edit access to maintainers (instructions)
  • Secrets in the connector's spec are annotated with airbyte_secret
  • Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
  • Code reviews completed
  • Documentation updated
    • Connector's README.md
    • Connector's bootstrap.md. See description and examples
    • docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
    • docs/integrations/README.md
    • airbyte-integrations/builds.md
  • PR name follows PR naming conventions

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

  • Create a non-forked branch based on this PR and test the below items on it
  • Build is successful
  • If new credentials are required for use in CI, add them to GSM. Instructions.
  • /test connector=connectors/<name> command is passing
  • New Connector version released on Dockerhub by running the /publish command described here
  • After the connector is published, connector added to connector index as described here
  • Seed specs have been re-generated by building the platform and committing the changes to the seed spec files, as described here
Updating a connector

Community member or Airbyter

  • Grant edit access to maintainers (instructions)
  • Secrets in the connector's spec are annotated with airbyte_secret
  • Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
  • Code reviews completed
  • Documentation updated
    • Connector's README.md
    • Connector's bootstrap.md. See description and examples
    • Changelog updated in docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
  • PR name follows PR naming conventions

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

  • Create a non-forked branch based on this PR and test the below items on it
  • Build is successful
  • If new credentials are required for use in CI, add them to GSM. Instructions.
  • /test connector=connectors/<name> command is passing
  • New Connector version released on Dockerhub and connector version bumped by running the /publish command described here
Connector Generator
  • Issue acceptance criteria met
  • PR name follows PR naming conventions
  • If adding a new generator, add it to the list of scaffold modules being tested
  • The generator test modules (all connectors with -scaffold in their name) have been updated with the latest scaffold by running ./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates then checking in your changes
  • Documentation which references the generator is updated as needed

@CLAassistant
Copy link

CLAassistant commented Mar 9, 2023

CLA assistant check
All committers have signed the CLA.

@midavadim midavadim linked an issue Mar 9, 2023 that may be closed by this pull request
@midavadim
Copy link
Contributor Author

midavadim commented Mar 9, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4374822516
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4374822516
🐛 https://gradle.com/s/rof3viiwjwj5a

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestConnection::test_check[inputs0] - AssertionError: as...
FAILED test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contai...
FAILED test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_refs_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - doc...
ERROR test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
=================== 5 failed, 28 passed, 4 errors in 23.90s ====================

@midavadim midavadim requested a review from lazebnyi March 9, 2023 13:16
@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Mar 9, 2023
@midavadim
Copy link
Contributor Author

midavadim commented Mar 9, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4376876322
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4376876322
🐛 https://gradle.com/s/e3pqtevoymtqe

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contai...
FAILED test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
ERROR test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - doc...
ERROR test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
=================== 4 failed, 31 passed, 2 errors in 49.89s ====================

@midavadim
Copy link
Contributor Author

midavadim commented Mar 10, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4383679153
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4383679153
🐛 https://gradle.com/s/g2vqxhccgista

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestConnection::test_check[inputs0] - AssertionError: as...
FAILED test_core.py::TestDiscovery::test_discover[inputs0] - docker.errors.Co...
FAILED test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contai...
FAILED test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - doc...
ERROR test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
=================== 6 failed, 28 passed, 3 errors in 20.07s ====================

@midavadim
Copy link
Contributor Author

midavadim commented Mar 10, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4386091840
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4386091840
🐛 https://gradle.com/s/zcgitlfjon5gk

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contai...
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
FAILED test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - doc...
============== 4 failed, 31 passed, 2 errors in 62.61s (0:01:02) ===============

@midavadim
Copy link
Contributor Author

midavadim commented Mar 10, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4386253311
✅ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4386253311
Python tests coverage:

Name                            Stmts   Miss  Cover
---------------------------------------------------
source_zoho_crm/exceptions.py       4      0   100%
source_zoho_crm/__init__.py         2      0   100%
source_zoho_crm/types.py          166      7    96%
source_zoho_crm/api.py             54      4    93%
source_zoho_crm/auth.py            17      2    88%
source_zoho_crm/source.py          14      5    64%
source_zoho_crm/streams.py        109     43    61%
---------------------------------------------------
TOTAL                             366     61    83%
Name                            Stmts   Miss  Cover
---------------------------------------------------
source_zoho_crm/exceptions.py       4      0   100%
source_zoho_crm/__init__.py         2      0   100%
source_zoho_crm/types.py          166      4    98%
source_zoho_crm/auth.py            17      2    88%
source_zoho_crm/api.py             54      8    85%
source_zoho_crm/streams.py        109     27    75%
source_zoho_crm/source.py          14      5    64%
---------------------------------------------------
TOTAL                             366     46    87%

Build Passed

Test summary info:

All Passed

@midavadim midavadim requested a review from brianjlai March 10, 2023 19:49
@midavadim
Copy link
Contributor Author

@YowanR
This connector is not eligible for low code becuase

  • it has dynamic schemas
  • current implementation support multithreading
    @lazebnyi

@YowanR
Copy link
Contributor

YowanR commented Mar 15, 2023

@midavadim Let's just enable it in Cloud without certifying to Beta, please 🙏

@midavadim midavadim requested a review from erohmensing March 15, 2023 14:09
@bnchrch bnchrch requested review from bnchrch and removed request for erohmensing March 15, 2023 16:31
@bnchrch
Copy link
Contributor

bnchrch commented Mar 16, 2023

Hey @midavadim would you be able to resolve the conflicts? Then im happy to review :)

…cloud

# Conflicts:
#	airbyte-integrations/connectors/source-zoho-crm/Dockerfile
#	airbyte-integrations/connectors/source-zoho-crm/acceptance-test-config.yml
#	airbyte-integrations/connectors/source-zoho-crm/setup.py
#	airbyte-integrations/connectors/source-zoho-crm/source_zoho_crm/auth.py
#	airbyte-integrations/connectors/source-zoho-crm/source_zoho_crm/spec.json
#	docs/integrations/sources/zoho-crm.md
@midavadim
Copy link
Contributor Author

midavadim commented Mar 16, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4441816580
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4441816580
🐛 https://gradle.com/s/7hje5swpwzxcc

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestConnection::test_check[inputs0] - AssertionError: as...
FAILED test_core.py::TestDiscovery::test_discover[inputs0] - docker.errors.Co...
FAILED test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - As...
FAILED test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contai...
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
FAILED test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_refs_exist_in_schema[inputs0]
=================== 7 failed, 28 passed, 2 errors in 50.68s ====================

@midavadim
Copy link
Contributor Author

midavadim commented Mar 16, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4441962090
✅ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4441962090
Python tests coverage:

Name                            Stmts   Miss  Cover
---------------------------------------------------
source_zoho_crm/exceptions.py       4      0   100%
source_zoho_crm/__init__.py         2      0   100%
source_zoho_crm/types.py          179     10    94%
source_zoho_crm/api.py             54      4    93%
source_zoho_crm/auth.py            17      2    88%
source_zoho_crm/source.py          14      5    64%
source_zoho_crm/streams.py        109     43    61%
---------------------------------------------------
TOTAL                             379     64    83%
Name                            Stmts   Miss  Cover
---------------------------------------------------
source_zoho_crm/exceptions.py       4      0   100%
source_zoho_crm/__init__.py         2      0   100%
source_zoho_crm/types.py          179      7    96%
source_zoho_crm/auth.py            17      2    88%
source_zoho_crm/api.py             54      8    85%
source_zoho_crm/streams.py        109     27    75%
source_zoho_crm/source.py          14      5    64%
---------------------------------------------------
TOTAL                             379     49    87%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:509: The previous and actual discovered catalogs are identical.
================== 36 passed, 1 skipped in 237.34s (0:03:57) ===================

@midavadim
Copy link
Contributor Author

Hey @midavadim would you be able to resolve the conflicts? Then im happy to review :)

I have updated this PR again

Copy link
Contributor

@bnchrch bnchrch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CAT passing, unit tests passing, incremental streams have source defined cursors.

Looks good to me! 👍

@midavadim
Copy link
Contributor Author

midavadim commented Mar 17, 2023

/publish connector=connectors/source-zoho-crm

🕑 Publishing the following connectors:
connectors/source-zoho-crm
https://github.com/airbytehq/airbyte/actions/runs/4449676917


Connector Did it publish? Were definitions generated?
connectors/source-zoho-crm

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@midavadim
Copy link
Contributor Author

midavadim commented Mar 17, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4449815172
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4449815172
🐛 https://gradle.com/s/ih23atredbtle

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestConnection::test_check[inputs0] - AssertionError: as...
FAILED test_core.py::TestDiscovery::test_discover[inputs0] - docker.errors.Co...
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_refs_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-allOf]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-not]
ERROR test_core.py::TestDiscovery::test_primary_keys_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_streams_has_sync_modes[inputs0] - doc...
ERROR test_core.py::TestDiscovery::test_additional_properties_is_true[inputs0]
ERROR test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - doc...
ERROR test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contain...
ERROR test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
ERROR test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
ERROR test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
ERROR test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
=================== 2 failed, 22 passed, 13 errors in 23.57s ===================

@midavadim
Copy link
Contributor Author

midavadim commented Mar 17, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4450009346
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4450009346
🐛 https://gradle.com/s/ewbpyd7zrwloi

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestConnection::test_check[inputs0] - AssertionError: as...
FAILED test_core.py::TestDiscovery::test_discover[inputs0] - docker.errors.Co...
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_refs_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-allOf]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-not]
ERROR test_core.py::TestDiscovery::test_primary_keys_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_streams_has_sync_modes[inputs0] - doc...
ERROR test_core.py::TestDiscovery::test_additional_properties_is_true[inputs0]
ERROR test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - doc...
ERROR test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contain...
ERROR test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
ERROR test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
ERROR test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
ERROR test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
=================== 2 failed, 22 passed, 13 errors in 23.21s ===================

@YowanR
Copy link
Contributor

YowanR commented Mar 17, 2023

This PR looks good, right? Can we add the ready to ship label?

@midavadim
Copy link
Contributor Author

midavadim commented Mar 19, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4461120614
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4461120614
🐛 https://gradle.com/s/n54b7a7muqpzi

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestConnection::test_check[inputs0] - AssertionError: as...
FAILED test_core.py::TestDiscovery::test_discover[inputs0] - docker.errors.Co...
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_refs_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-allOf]
ERROR test_core.py::TestDiscovery::test_defined_keyword_exist_in_schema[inputs0-not]
ERROR test_core.py::TestDiscovery::test_primary_keys_exist_in_schema[inputs0]
ERROR test_core.py::TestDiscovery::test_streams_has_sync_modes[inputs0] - doc...
ERROR test_core.py::TestDiscovery::test_additional_properties_is_true[inputs0]
ERROR test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - doc...
ERROR test_core.py::TestBasicRead::test_read[inputs0] - docker.errors.Contain...
ERROR test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
ERROR test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
ERROR test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
ERROR test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
=================== 2 failed, 22 passed, 13 errors in 23.28s ===================

@midavadim
Copy link
Contributor Author

midavadim commented Mar 19, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4461228467
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4461228467
🐛 https://gradle.com/s/aoc7bwfs55fsk

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestDiscovery::test_discover[inputs0] - docker.errors.Co...
FAILED test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - As...
FAILED test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
FAILED test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
ERROR test_core.py::TestBasicRead::test_read[inputs0] - Failed: High strictne...
==================== 6 failed, 30 passed, 1 error in 22.74s ====================

@midavadim
Copy link
Contributor Author

midavadim commented Mar 19, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4461315705
❌ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4461315705
🐛 https://gradle.com/s/ql5ajfkwjrvmi

Build Failed

Test summary info:

=========================== short test summary info ============================
FAILED test_core.py::TestConnection::test_check[inputs0] - AssertionError: as...
FAILED test_core.py::TestDiscovery::test_discover[inputs0] - docker.errors.Co...
FAILED test_core.py::TestDiscovery::test_backward_compatibility[inputs0] - As...
FAILED test_full_refresh.py::TestFullRefresh::test_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_two_sequential_reads[inputs0]
FAILED test_incremental.py::TestIncremental::test_read_sequential_slices[inputs0]
FAILED test_incremental.py::TestIncremental::test_state_with_abnormally_large_values[inputs0]
ERROR test_core.py::TestDiscovery::test_defined_cursors_exist_in_schema[inputs0]
=============== 7 failed, 29 passed, 1 error in 70.13s (0:01:10) ===============

@midavadim
Copy link
Contributor Author

midavadim commented Mar 19, 2023

/test connector=connectors/source-zoho-crm

🕑 connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4462361939
✅ connectors/source-zoho-crm https://github.com/airbytehq/airbyte/actions/runs/4462361939
Python tests coverage:

Name                            Stmts   Miss  Cover
---------------------------------------------------
source_zoho_crm/exceptions.py       4      0   100%
source_zoho_crm/__init__.py         2      0   100%
source_zoho_crm/types.py          179     10    94%
source_zoho_crm/api.py             54      4    93%
source_zoho_crm/auth.py            17      2    88%
source_zoho_crm/source.py          14      5    64%
source_zoho_crm/streams.py        109     43    61%
---------------------------------------------------
TOTAL                             379     64    83%
Name                            Stmts   Miss  Cover
---------------------------------------------------
source_zoho_crm/exceptions.py       4      0   100%
source_zoho_crm/__init__.py         2      0   100%
source_zoho_crm/types.py          179      7    96%
source_zoho_crm/auth.py            17      2    88%
source_zoho_crm/api.py             54      8    85%
source_zoho_crm/streams.py        109     27    75%
source_zoho_crm/source.py          14      5    64%
---------------------------------------------------
TOTAL                             379     49    87%

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:509: The previous and actual discovered catalogs are identical.
================== 36 passed, 1 skipped in 229.84s (0:03:49) ===================

@midavadim
Copy link
Contributor Author

/pblish connector=connectors/source-zoho-crm

@midavadim
Copy link
Contributor Author

midavadim commented Mar 19, 2023

/publish connector=connectors/source-zoho-crm

🕑 Publishing the following connectors:
connectors/source-zoho-crm
https://github.com/airbytehq/airbyte/actions/runs/4462607877


Connector Did it publish? Were definitions generated?
connectors/source-zoho-crm

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@midavadim midavadim merged commit f2ba087 into master Mar 19, 2023
@midavadim midavadim deleted the midavadim/23484-source-zoho-crm-enable-on-cloud branch March 19, 2023 21:24
@YowanR
Copy link
Contributor

YowanR commented Mar 21, 2023

This PR was merged but the connector is not on Cloud at the moment. Can you look into this @midavadim @lazebnyi ?

erohmensing pushed a commit that referenced this pull request Mar 22, 2023
* Fixed authentication to support latest CDK, fixed SAT, updated expected record, enabled high strictness

* updated docs and connector version

* updated format of abnormal state

* updated version is seed

* merged

* updated connector version

* fixed SAT

* fixed SAT

* added empty streams

* auto-bump connector version

---------

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
erohmensing pushed a commit that referenced this pull request Mar 22, 2023
* Fixed authentication to support latest CDK, fixed SAT, updated expected record, enabled high strictness

* updated docs and connector version

* updated format of abnormal state

* updated version is seed

* merged

* updated connector version

* fixed SAT

* fixed SAT

* added empty streams

* auto-bump connector version

---------

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/zoho-crm ready-to-ship
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Source ZohoCRM: Enable in Cloud
6 participants