c2corg / v6_ui

UI for c2c.org v6
GNU Affero General Public License v3.0
7 stars 12 forks source link

New outing button in waypoint detail view page #375

Closed asaunier closed 8 years ago

asaunier commented 8 years ago

I have noticed there is a "add a new outing to this document" button in the list of recents outings in a WP detail view page.

I am not sure it is relevant since it opens the outing creation form with a GET parameter ?waypoints=123456 => but it doesn't really help choosing a route to be associated to the outing.

On the other hand it would make sense to show an "add route to this WP" button in the WP detail view pages (as in v5).

In v5

The "add an outing" button is only available in the route detail view page, not in the summit detail view pages. For instance: http://www.camptocamp.org/summits/40576/fr/dome-des-glaciers On the other hand the summit detail view page offers a "add a route" button.

Climbing outdoor sites are an exception in v5:

This is a quite complicated topic...

fjacon commented 8 years ago

Also see : https://github.com/c2corg/v6_ui/issues/402#issuecomment-234656396

asaunier commented 8 years ago

Having outings directly associated to climbing sites as in v5 is not possible. And not a good thing because it would create another exception. It's better to keep the standard relationships between pages: waypoints <> routes <> outings

Here is a possible workflow for v6:

By the way: in v5, it is not possible to create outings associated to indoor climbing_sites. In v6, I suggest to drop this restriction: same behaviour for all WP (summits, accesses, indoor climbing sites, outdoor climbing sites, etc.).

stef74 commented 8 years ago

For migration : created generic route For outing : route needed No direct outing with WP

loicperrin commented 8 years ago

By the way: in v5, it is not possible to create outings associated to indoor climbing_sites

In V6 it will be possible to create routes associated to each type of WP (even indoor climbing_sites), and then to create outings on this routes.

asaunier commented 8 years ago

Closing this issue because: