Skip to content
This repository has been archived by the owner on Jul 29, 2020. It is now read-only.

Metadata sync #244

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Metadata sync #244

wants to merge 18 commits into from

Conversation

wasade
Copy link
Member

@wasade wasade commented Aug 21, 2017

This is good for first round review. Constants are being evaluated out-of-band, please do not merge until I provide a 👍 on that.

cc @josenavas @EmbrietteH

Copy link
Member

@josenavas josenavas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good - just one comment!

@@ -687,6 +688,9 @@ def format_survey_data(self, md, external_surveys=None, full=False): # noqa
list of tuple of str
The barcode and error message if something failed
"""
not_provided = 'Not provided'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wasade
Copy link
Member Author

wasade commented Aug 22, 2017 via email

@EmbrietteH
Copy link
Contributor

👍
Any idea why the Travis build failed?

@josenavas
Copy link
Member

Thanks for the info @wasade! It is weird as that is the value listed in the website to use. @antgonza any input on why there was so many problems using those values?

@wasade
Copy link
Member Author

wasade commented Aug 22, 2017 via email

@wasade
Copy link
Member Author

wasade commented Aug 22, 2017 via email

@wasade
Copy link
Member Author

wasade commented Aug 22, 2017

Some failures are due tests that I didn't catch to fix. Others look unrelated. Updating.

barcode['COUNTRY'] = country_lookup[country]
barcode['GEO_LOC_NAME'] = ':'.join(
[barcode['COUNTRY'], barcode['STATE']])
#except KeyError:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see note in _integrate_barcode_information

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants