Skip to content
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

implement full indieweb/mf2 location algorithm #15

Merged
merged 1 commit into from
Oct 27, 2016

Conversation

snarfed
Copy link
Collaborator

@snarfed snarfed commented Oct 26, 2016

...to collect properties from location, adr, geo, and top-level objects.

https://indieweb.org/location#How_to_determine_the_location_of_a_microformat

for snarfed/bridgy#706. used in snarfed/granary@55f19cd.

assuming you're ok with this, i don't feel strongly about whether we cut a new release for it or not. i'm fine with pointing granary/bridgy's reqt's to the github repo. either way.

thanks in advance!

snarfed added a commit to snarfed/granary that referenced this pull request Oct 26, 2016
fixes snarfed/bridgy#706

uses kylewm/mf2util#15

TODO: update requirements.txt to point to either a new mf2util release, if we do one, or the github repo at that commit.
@kylewm kylewm merged commit b0b6b5c into kylewm:master Oct 27, 2016
@kylewm
Copy link
Owner

kylewm commented Oct 27, 2016

Whoa this is awesome. Thank you for doing it!

@kylewm
Copy link
Owner

kylewm commented Oct 27, 2016

Went ahead and uploaded a new release https://pypi.python.org/pypi/mf2util/0.5.0 👍

@snarfed
Copy link
Collaborator Author

snarfed commented Oct 27, 2016

thanks for merging!

@snarfed
Copy link
Collaborator Author

snarfed commented Oct 27, 2016

oh btw you'll probably want to push your 0.5.0 version bump commit when you get a chance. not urgent. :P

@snarfed
Copy link
Collaborator Author

snarfed commented Nov 1, 2016

oh btw you'll probably want to push your 0.5.0 version bump commit when you get a chance. not urgent. :P

friendly ping @kylewm!

@kylewm
Copy link
Owner

kylewm commented Nov 1, 2016

Ohh thanks for the ping! Missed your first message (as you can probably tell)

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

Successfully merging this pull request may close these issues.

2 participants