-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix Chile Compra Bulk #437
Conversation
Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
…e into 375-chile-bulk-fix Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
…e into 375-chile-bulk-fix
Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
'errors': json_data, | ||
}) | ||
else: | ||
return super().build_file(data=data, file_name=file_name, url=url, data_type=data_type, encoding=encoding) |
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 we instead do data=None, **kwargs
in the method signature, and then super().build_file(data=data, **kwargs)
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.
I think it is kind of the same, but I changed it
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.
Indeed – but this way we don't need to change two places if we add keyword arguments :)
@@ -67,5 +73,6 @@ def parse_list(self, response): | |||
meta={ | |||
'year': year, | |||
'month': month, | |||
} | |||
}, | |||
callback=self.parse_list |
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.
@jpmckinney sorry, I noticed that chile_base was missing a callback call in the parse_list method so I included the fix here too
Signed-off-by: Yohanna Lisnichuk <yohanitalisnichuk@gmail.com>
closes #375 #408