https://kedro.org/ logo
#random
Title
# random
f

FlorianGD

08/10/2023, 3:07 PM
Hi, I found a bug in the
api.APIDataSet
in
kedro-datasets
: the data is doubly encoded as json, making it a string of a string. We should just let the
requests
library do the conversion for us. I created this PR to fix it https://github.com/kedro-org/kedro-plugins/pull/301. It seems that it must be validated by a maintener, for the CI to run
❤️ 5
n

Nok Lam Chan

08/10/2023, 3:17 PM
Hey @FlorianGD, thanks for flagging this and your contributions always : )
🙏 1
I have triggered the run already (We moved to Github Action recently and the default requires so)
👍 1
f

FlorianGD

08/10/2023, 3:40 PM
There are some linting errors, some of which do not seem directly related to the changes I have made. Do you want me to take a look also, or is it addressed in another PR/branch?
It was really small, I made the change, it should pass now