elodie icon indicating copy to clipboard operation
elodie copied to clipboard
verified publisher icon jmathai

Add human readable location data to picture metadata

Open bdillahu opened this issue 9 years ago • 6 comments

Proposed addition to address Issue #195

Adds an option to enable this feature.

bdillahu avatar Feb 07 '17 23:02 bdillahu

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Feb 07 '17 23:02 CLAassistant

Coverage Status

Coverage decreased (-0.7%) to 92.155% when pulling 146bf0d80a54ab12af68cad7b7aea1e35c7247a6 on bdillahu:location into 56f0b1e170e28ef29e0ed439e7ca0e4ea1b93acb on jmathai:master.

coveralls avatar Feb 07 '17 23:02 coveralls

Sorry for delay... kind of swamped with other things.

I don't disagree that it needs tests added, but not sure I have the skills/understanding/time. If I get that far, I will, but wanted to at least get you this back.

bdillahu avatar Mar 28 '17 22:03 bdillahu

Coverage Status

Coverage decreased (-0.5%) to 92.372% when pulling b033d32e898abc44cf0e97e0bb6d563f3491da51 on bdillahu:location into 56f0b1e170e28ef29e0ed439e7ca0e4ea1b93acb on jmathai:master.

coveralls avatar Mar 28 '17 22:03 coveralls

Coverage Status

Coverage decreased (-0.5%) to 92.372% when pulling b033d32e898abc44cf0e97e0bb6d563f3491da51 on bdillahu:location into 56f0b1e170e28ef29e0ed439e7ca0e4ea1b93acb on jmathai:master.

coveralls avatar Mar 28 '17 22:03 coveralls

@bdillahu understandable about tests. I'll see if I can get to tests as well. Always appreciate PRs submitted early so they can be iterated upon.

jmathai avatar Apr 13 '17 06:04 jmathai