Skip to content

Conversation

@bidoubiwa
Copy link
Contributor

No description provided.

Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

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

Great work 🔥 !
Just ask few questions

@bidoubiwa bidoubiwa requested a review from alallema October 13, 2021 18:41
alallema
alallema previously approved these changes Oct 14, 2021
Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

@alallema
Copy link
Contributor

It works well 🔥
Just:

  • Change the icon before Geo Search in the README
  • Modify the sentence on the index.html about the ~20K european countries.

Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

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

Well Done 💪

@bidoubiwa
Copy link
Contributor Author

bors merge

@bors
Copy link
Contributor

bors bot commented Oct 18, 2021

Build succeeded:

@bors bors bot merged commit 8c78c31 into main Oct 18, 2021
@bors bors bot deleted the geo_search branch October 18, 2021 14:27
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.

3 participants