The form validation for the Carto URL field expects there to be a '/builder/' path component. So urls like https://plopesc.carto.com/builder/dc610160-30b0-11e7-b058-0e233c30368f/e... work fine. But Carto also provides embeddable maps with a '/viz/' path component instead of '/builder/'. This module embeds and displays the '/viz/' maps just fine. The '/viz'/ urls just fail validation.
The validation function should be updated to allow either both 'builder' and 'viz', or (my preference) allow any single word in that path spot.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | allow_user_path_components-3032267-4.patch | 625 bytes | ochenk |
| #2 | abstract_path_component-3032267-2.patch | 609 bytes | ochenk |
Comments
Comment #2
ochenk commentedThis patch abstracts the '/builder/' path component validation to accept any single alpha word.
Comment #3
ochenk commentedComment #4
ochenk commentedCarto also publishes maps with usernames as path components. For example, a map can be available at
https://client.carto.com/u/username/viz/72159080-31f9-4509-a018-217a6d99...
and at
https://client.carto.com/viz/72159080-31f9-4509-a018-217a6d99c752/embed_map
This patch supports optional username components as well.
Comment #6
plopescPatch applied and new release published. thank you!
Comment #8
plopesc