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

GeoENS pull request - initial revision #2390

Merged
merged 11 commits into from Sep 13, 2020

Conversation

james-choncholas
Copy link
Contributor

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md

We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

  • The PR edits only existing draft PRs.
  • The build passes.
  • Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
  • If matching on email address, the email address is the one publicly listed on your GitHub profile.

@axic axic added the type: ERC label Nov 25, 2019
james-choncholas added a commit to james-choncholas/geoens-dev that referenced this pull request Dec 3, 2019
james-choncholas added a commit to james-choncholas/resolvers that referenced this pull request May 8, 2020
GeoENS brings geographic split horizon capabilities to ENS.
See more at EIP 2390 ethereum/EIPs#2390
james-choncholas added a commit to james-choncholas/resolvers that referenced this pull request May 8, 2020
GeoENS brings geographic split horizon capabilities to ENS.
See more at EIP 2390 ethereum/EIPs#2390
james-choncholas added a commit to james-choncholas/resolvers that referenced this pull request May 8, 2020
GeoENS brings geographic split horizon capabilities to ENS.
See more at EIP 2390 ethereum/EIPs#2390
james-choncholas added a commit to james-choncholas/resolvers that referenced this pull request May 8, 2020
GeoENS brings geographic split horizon capabilities to ENS.
See more at EIP 2390 ethereum/EIPs#2390
@github-actions
Copy link

github-actions bot commented Sep 8, 2020

There has been no activity on this pull request for two months. It will be closed in a week if no further activity occurs. If you would like to move this EIP forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@github-actions github-actions bot added the stale label Sep 8, 2020
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
james-choncholas and others added 2 commits September 9, 2020 12:48
Co-authored-by: Micah Zoltu <micah@zoltu.net>
Flatten references and point discussion to new GitHub Issue
@james-choncholas
Copy link
Contributor Author

Thank you for your feedback! I appreciate your interest in this idea and have incorporated all your suggestions

@github-actions github-actions bot removed the stale label Sep 9, 2020
Copy link
Contributor

@MicahZoltu MicahZoltu left a comment

Choose a reason for hiding this comment

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

Missed these in my previous review.

EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
Co-authored-by: Micah Zoltu <micah@zoltu.net>
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Show resolved Hide resolved
Add security considerations
Improve formatting and add note about different types of geohash
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
EIPS/eip-2390.md Outdated Show resolved Hide resolved
Comment on lines +313 to +314


Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

For repository wide consistency.

@MicahZoltu MicahZoltu merged commit 45019dc into ethereum:master Sep 13, 2020
tkstanczak pushed a commit to tkstanczak/EIPs that referenced this pull request Nov 7, 2020
GeoENS brings geographic split horizon capabilities to ENS. It's GeoDNS for ENS!
Arachnid pushed a commit to Arachnid/EIPs that referenced this pull request Mar 6, 2021
GeoENS brings geographic split horizon capabilities to ENS. It's GeoDNS for ENS!
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.

None yet

3 participants