additional_data: sources: add licence info#335
Conversation
af01824 to
fda34e0
Compare
| self._cache = {} | ||
|
|
||
| def update_additional_data(self, grant, source_file, additional_data): | ||
| additional_data[f"{self.ADDITIONAL_DATA_KEY}_LICENCE"] = self.LICENCE |
There was a problem hiding this comment.
I'm not sure why we're doing _LICENCE here, compared to https://github.com/ThreeSixtyGiving/datastore/pull/258/changes#diff-d3929aef8ca975fa331395aa7d04c5dc0b85608c6aed7ab04f3a45cf32fa4095R15-R22 which is additional_data[self.ADDITIONAL_DATA_KEY]["source_license"].
b56608d to
830d3fc
Compare
830d3fc to
7fa3b96
Compare
|
Not sure how to test so asked for Matt's review instead |
Bjwebb
left a comment
There was a problem hiding this comment.
This is looking good, if the outputted JSON is what we want. Could you share an example grant JSON? I think that might help me and @mrshll1001 review this.
|
I have added print statements in the test. The output: |
Bjwebb
left a comment
There was a problem hiding this comment.
I think if you can confirm that output is correct, the code looks good.
|
Thanks @Bjwebb . @mrshll1001 is it okay for you? |
relates to #213
I have added the licence to the additional data sources.
The API will be done in a different PR.