Merge lp://staging/~merkurie/sahana-eden/potlatch into lp://staging/sahana-eden

Proposed by Jackie Lai
Status: Needs review
Proposed branch: lp://staging/~merkurie/sahana-eden/potlatch
Merge into: lp://staging/sahana-eden
Diff against target: 4030 lines (+3635/-358)
3 files modified
static/potlatch2/map_features/shopping.xml (+1/-183)
static/potlatch2/map_features/tourism.xml (+1/-175)
static/potlatch2/map_features_eden.xml (+3633/-0)
To merge this branch: bzr merge lp://staging/~merkurie/sahana-eden/potlatch
Reviewer Review Type Date Requested Status
Fran Boon Needs Fixing
Review via email: mp+82259@code.staging.launchpad.net

Description of the change

remove some unnecessary map features.

To post a comment you must log in.
Revision history for this message
Fran Boon (flavour) wrote :

Wouldn't the new file 'map_features_eden.xml' need to be referenced from somewhere?

In general I'd prefer to comment out lines which are inappropriate rather than remove them entirely - both in the main 'map_features.xml' & in the included files.
(Our ideas of what are relevant may not always match with what other deployments wish to do)
We probably also want one new included page which would be the topmost tab to cover those features which we see as most relevant to this domain - starting with those identified in the existing HOT presets file:
http://pierzen.dev.openstreetmap.org/hotpotlatch2/potlatch2.html
& ideas from here:
http://wiki.openstreetmap.org/wiki/Humanitarian_OSM_Tags

Many thanks :)
Fran.

review: Needs Fixing

Unmerged revisions

2874. By dev <dev@eden-dev>

grabbed flavour's potlatch update and cleared out some unnecessary map features.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.