airbytehq / airbyte

The leading data integration platform for ETL / ELT data pipelines from APIs, databases & files to data warehouses, data lakes & data lakehouses. Both self-hosted and Cloud-hosted.
https://airbyte.com
Other
16.07k stars 4.11k forks source link

:sparkles: Source Yandex Metrica : Migrate Python CDK to Low-code CDK #30900

Open bmcgilli opened 1 year ago

bmcgilli commented 1 year ago

https://docs.airbyte.com/integrations/sources/yandex-metrica

Rules for the Connectors Hackathon You can only work on one connector at a time Make sure to resolve all suggestions from the Airbyte team Must include all streams of the connector you’re migrating. Otherwise, it will not be accepted. If you don’t commit anything within five days, the issue may be assigned to someone else.

donjne commented 1 year ago

@bmcgilli Can I work on this?

justinbchau commented 1 year ago

@donjne I have already assigned you an issue so I will leave this open to give others a chance!

if you raise a PR and this issue is still open, let me know and I can assign this one to you!

bishalbera commented 1 year ago

@justinbchau Hey can I give it a try?

bishalbera commented 1 year ago

@justinbchau I'm getting this error


  File "/home/bishal01/airbyte-linux/airbyte-integrations/connectors/source-yandex-metrica/.venv/lib/python3.11/site-packages/jsonschema/validators.py", line 934, in validate
    raise error
jsonschema.exceptions.ValidationError: 'TokenAuthenticator' is not one of ['ApiKeyAuthenticator']

Failed validating 'enum' in schema[0]['properties']['type']:
    {'enum': ['ApiKeyAuthenticator'], 'type': 'string'}

On instance['type']:
    'TokenAuthenticator'

   any thoughts?
bishalbera commented 1 year ago

@justinbchau Hey I'm facing the previous mentioned problem though in the python cdk its mentioned that it uses tokenauthenticator for like past 3 days . So would you please unassign me so that I can try solving other issues. ty

justinbchau commented 1 year ago

I will leave this assigned for now but tagged as blocked until we can figure out what is going on