-
-
Notifications
You must be signed in to change notification settings - Fork 33
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.
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
Show associated images on image view #396
Comments
The associated images don't appear in the linked docs : https://www.demov6.camptocamp.org/images/208579/fr/face-n-des-grandes-jorasses-quelques-itineraires The association has been migrated correctly ? |
For example for https://www.demov6.camptocamp.org/routes/557726/fr/grandes-jorasses-pointe-walker-colton-mcintyre your image is returned by the API: http://api.demov6.camptocamp.org/routes/557726 But it seems that the images are not shown. |
I verified that the images associations to an image have been migrated correctly. |
What association to which document are you missing? |
In https://api.demov6.camptocamp.org/images/188564, it misses the association to the image 208579, as they are associated each other, and there is no hierarchy in the image association. |
Thanks, I created an issue for the API: c2corg/v6_api#530 |
There is nothing in the image detail view template that would display the associated images (in an image carousel as for the other document types): |
There is two issues in one : API does not shows association A > B if association B > A exists. : c2corg/v6_api#530 And a gallery is missing if associations exists. Let's go with this : this issue will describe the UI part, and API issue will describe API part |
Show associated images on image view #396
In v5 it's possible to associate an image to another (for instance to associate a drawing with the original image). For instance:
http://www.camptocamp.org/images/208579/fr/face-n-des-grandes-jorasses-quelques-itineraires
http://www.camptocamp.org/images/188564/fr/face-nord-des-grandes-jorasses
As for now I don't think it's possible in v6.
The text was updated successfully, but these errors were encountered: