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.

Sign up for GitHub

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

Jump to bottom

What3Words API v3 #444

Merged
merged 3 commits into from Mar 21, 2021
Merged

What3Words API v3 #444

merged 3 commits into from Mar 21, 2021

Conversation

saidtezel
Copy link
Contributor

@saidtezel saidtezel commented Sep 9, 2020 •
edited by KostyaEsmukov

Added new class and associated tests for the What3Words API v3.

Supersedes #434

@KostyaEsmukov KostyaEsmukov added this to the 2.2 milestone Dec 20, 2020
KostyaEsmukov
KostyaEsmukov approved these changes Mar 21, 2021
View reviewed changes
Copy link
Member

@KostyaEsmukov KostyaEsmukov left a comment

Choose a reason for hiding this comment

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

Thanks! Sorry for waiting for so long. Gonna make some changes on top of that (mainly -- collapse what3words.py and what3wordsv3.py to a single module and reuse tests via inheritance).

@KostyaEsmukov KostyaEsmukov merged commit 08d5605 into geopy:master Mar 21, 2021
KostyaEsmukov added a commit that referenced this pull request Mar 21, 2021
@KostyaEsmukov
What3WordsV3: update lang doc url (#444)
6b8275b
KostyaEsmukov added a commit that referenced this pull request Mar 21, 2021
@KostyaEsmukov
What3WordsV3: move to what3words.py (#444)
4424c17
KostyaEsmukov added a commit that referenced this pull request Apr 10, 2021
@KostyaEsmukov
docs what3words: don't use the word current as it can change later (#…
8b88e29 
…444)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KostyaEsmukov KostyaEsmukov KostyaEsmukov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.2
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@saidtezel @KostyaEsmukov