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

restrict geocoding/search to current country #166

Closed
crijke opened this issue Mar 19, 2017 · 3 comments
Closed

restrict geocoding/search to current country #166

crijke opened this issue Mar 19, 2017 · 3 comments
Assignees
@sjockers @crijke
Labels
blocked

Comments

@crijke
Copy link
Member

crijke commented Mar 19, 2017 •
edited
Loading

should the search really be restricted to the currently selected country? or is this too confusing for the user? also, should it just show places of the currently selected country or all places?

how does the geocoder behave when adding an entry? should it also be restricted to just one country?

maybe it's more intuitive to always show results from all supported countries (CH/DE)
@crijke crijke added the blocked label Mar 19, 2017
@sjockers
Copy link
Member

should the search really be restricted to the currently selected country? or is this too confusing for the user?

Very good point. If it is easy, we should go with searching in both countries for now.

also, should it just show places of the currently selected country or all places?

We should always show all places.

how does the geocoder behave when adding an entry? should it also be restricted to just one country? maybe it's more intuitive to always show results from all supported countries (CH/DE)

Yes, let's go with both countries for now.

@crijke
Copy link
Member Author

crijke commented Mar 19, 2017 •
edited
Loading

Restricting search to multiple countries is currently not possible, but we could use a bounding box that comes close:

bounding_box_tool__metadata_enrichment_for_catalogue_records_by_visually_selecting_geographic_coordinates__latitude___longitude__for_maps

@sjockers
Copy link
Member

Ok, let's go for this.

@crijke crijke closed this as completed Mar 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sjockers sjockers

@crijke crijke

Labels
blocked
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sjockers @crijke