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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Source Monday: fix empty activity logs extractor #35696
Conversation
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
The latest updates on your projects. Learn more about Vercel for Git 鈫楋笌 1 Ignored Deployment
|
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@@ -41,9 +41,8 @@ def extract_records(self, response: requests.Response) -> List[Record]: | |||
return result | |||
|
|||
for board_data in response_body["data"]["boards"]: | |||
if not isinstance(board_data, dict): | |||
if not isinstance(board_data, dict) or not board_data.get("activity_logs"): |
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.
Should there be a unit test for the case when response_body["data"]["boards"]
doesn't have the activity_logs
section?
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.
added
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
What
Resolve https://github.com/airbytehq/oncall/issues/4395
How
pass if
activity_logs
value isnull
Recommended reading order
airbyte-integrations/connectors/source-monday/source_monday/extractor.py
馃毃 User Impact 馃毃
no breaking changes
Pre-merge Actions
Updating a connector
Community member or Airbyter
Airbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.