Remove map url field from venues
Bug #524013 reported by
Thomas Bechtold
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
LoCo Team Portal |
Fix Released
|
High
|
Unassigned |
Bug Description
i think longitude and latitude are enough. we don't need the map url field.
Related branches
lp:~effie-jayx/loco-team-portal/toabctl_openlayer_merge
- Daniel Holbach (community): Approve
- Michael Hall (community): Approve
-
Diff: 406 lines (+221/-59)10 files modifiedloco_directory/media/css/newstyle.css (+15/-8)
loco_directory/media/js/common.js (+103/-0)
loco_directory/templates/base.html (+8/-17)
loco_directory/templates/venues/venue_detail.inc.html (+2/-2)
loco_directory/templates/venues/venue_detail_basic.inc.html (+22/-10)
loco_directory/templates/venues/venue_new.html (+0/-16)
loco_directory/templates/venues/venue_update.html (+23/-4)
loco_directory/venues/migrations/0004_remove_venue_map_field.py (+46/-0)
loco_directory/venues/models.py (+0/-1)
loco_directory/venues/views.py (+2/-1)
tags: | added: schema-change |
tags: | added: venues |
tags: | added: needs-decision |
Changed in loco-directory: | |
status: | New → Confirmed |
tags: | added: maps |
Changed in loco-directory: | |
importance: | Undecided → Low |
status: | Confirmed → Triaged |
Changed in loco-directory: | |
importance: | Low → High |
tags: | removed: needs-decision |
Changed in loco-directory: | |
milestone: | none → 0.2.11 |
status: | Triaged → Fix Committed |
Changed in loco-directory: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I personally find the maps much more useful than longitude and latitude. Longitude and Latitude mean absolutely nothing to me. I would simply copy them to Google Maps to figure out where the coordinates point to. However, I do agree that there is no point in specifying both longitude, latitude, and a map url. Instead, the map url should be generated automatically using the coordinates (or the other way around, generate the coordinates from the map url).