Groupe open/panoramax streetview support by nrc97 · Pull Request #11822 · geosolutions-it/MapStore2

nrc97 · 2025-12-16T11:34:53Z

Description

This PR introduces support for Panoramax as an additional Street View provider in MapStore2. Users can now visualize street‑level imagery using Panoramax alongside existing providers. The feature includes configuration options, i18n strings, attribution handling, and provider selection at runtime.

Please check if the PR fulfills these requirements

The commit message follows our guidelines: https://github.com/geosolutions-it/MapStore2/blob/master/CONTRIBUTING.md
Tests for the changes have been added (for bug fixes / features)
Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x", remove the others)

Feature

Issue

What is the current behavior?
Panoramax is not supportted as streetview provider

What is the new behavior?
We can now use panoramax as provider for the streetview plugin.

Breaking change

No

Other useful information

Panoramax OpenAPI Documentation: https://docs.panoramax.fr/api/api/openapi/
Panoramax Viewer Documentation: https://docs.panoramax.fr/web-viewer/#basic-usage

tdipisa · 2025-12-16T15:59:17Z

@nrc97 thank you so much for contributing with this. We will review as soon as possible.

map.forEachFeatureAtPixel in openlayers/Map.jsx adapted for detection of features of type RenderFeature used for MVT layers. By doing that, MVT layers are clickable and the intersected features are return on clic TileProviderLayer for being able to create MVT layer and display it.

- Create a viewer using pnx-photo-viewer to display pictures. - Add a Panoramax VectorTileLayer using OpenLayers styling to match other providers. - Handle click events: retrieve picture data directly from the layer datas, or call the API if the data is not found.

…h, Italian and German

tdipisa · 2025-12-17T17:18:17Z

Hi @nrc97 if you are still working on this, I kindly ask you to convert this PR to a Draft PR to avoid reviewing if you haven't completed yest the development. Thank you so much again for the collaboration and the contribution.

Gaetanbrl · 2025-12-18T09:03:33Z

Is this a tool dedicated to OpenLayers ? (Mapstore offering several map libraries)

landryb · 2026-01-06T15:55:50Z

Fwiw, i'm not in a position to judge the code itself, but from basic testing against a locally deployed panoramax instance and a locally running mapstore from this PR, this works fine, eg sequences are loaded as MVT from the api, and clicking on the sequence brings up a window showing the picture.

Features are basic (eg no link to the complete sequence, which can be useful, nor filters like on panoramax web viewer), but there's always the link to the original picture in panoramax, so that's fine.

if i had a user-level remark to do, when the streetview window has its default scale (eg 'smallish'), the 'back' arrow is a bit hidden by the licence footer for the image. one has to make the window a bit higher to easily interact with the arrow.

tdipisa · 2026-01-07T14:26:24Z

Hi @nrc97 if you are still working on this, I kindly ask you to convert this PR to a Draft PR to avoid reviewing if you haven't completed yest the development. Thank you so much again for the collaboration and the contribution.

@nrc97 can we finally schedule a review session for this? Thank you for confirming that.

nrc97 · 2026-01-13T08:51:20Z

Hi @nrc97 if you are still working on this, I kindly ask you to convert this PR to a Draft PR to avoid reviewing if you haven't completed yest the development. Thank you so much again for the collaboration and the contribution.

@nrc97 can we finally schedule a review session for this? Thank you for confirming that.

@tdipisa Yes, I'll send you my availability by email.

tdipisa · 2026-01-13T08:59:29Z

@tdipisa Yes, I'll send you my availability by email.

@nrc97 no worries, I was simply asking if we can proceed with a review. I was asking for confirmation that everything is ready for this. No need for a call.

Can you please fix conflicts? Thank you so much for contributing.

tdipisa

@nrc97 conflicts fix needed.

cla-bot · 2026-01-21T14:21:08Z

In order to contribute to the MapStore project, the CLA (Contributor License agreement) should be sent signed to GeoSolutions. Please consult contributing rules at: https://github.com/geosolutions-it/MapStore2/wiki/Contributing-to-MapStore#contributing-code

tdipisa · 2026-01-22T08:53:47Z

In order to contribute to the MapStore project, the CLA (Contributor License agreement) should be sent signed to GeoSolutions. Please consult contributing rules at: https://github.com/geosolutions-it/MapStore2/wiki/Contributing-to-MapStore#contributing-code

@nrc97 please notice that, thank you.

Removed unused DEFAULT_SRS constant from panoramax.js to match the eslint configuration

cla-bot · 2026-01-28T14:57:37Z

In order to contribute to the MapStore project, the CLA (Contributor License agreement) should be sent signed to GeoSolutions. Please consult contributing rules at: https://github.com/geosolutions-it/MapStore2/wiki/Contributing-to-MapStore#contributing-code

tdipisa · 2026-02-24T17:23:19Z

@nrc97 please can you sign the CLA as requested to contribute your code?

Copilot

Pull request overview

Adds Panoramax as a new provider for the StreetView plugin, including UI, API integration, and OpenLayers support for Panoramax MVT coverage tiles.

Changes:

Added Panoramax provider wiring (constants, panels, API module, epics integration) and new UI viewer component.
Added OpenLayers support for MVT-based tileprovider layers and improved feature-picking for MVT RenderFeatures.
Added i18n strings for Panoramax and introduced the @panoramax/web-viewer dependency.

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
web/client/translations/data.it-IT.json	Adds Panoramax “empty selection” message.
web/client/translations/data.fr-FR.json	Adds Panoramax “empty selection” message.
web/client/translations/data.es-ES.json	Adds Panoramax “empty selection” message.
web/client/translations/data.en-US.json	Adds Panoramax “empty selection” message.
web/client/translations/data.de-DE.json	Adds Panoramax “empty selection” message.
web/client/plugins/StreetView/StreetView.jsx	Documents Panoramax as a supported StreetView provider.
web/client/plugins/StreetView/selectors/streetView.js	Adds default Panoramax data-layer config (MVT tileprovider) and styling.
web/client/plugins/StreetView/epics/streetView.js	Passes provider settings into provider `getLocation` calls.
web/client/plugins/StreetView/containers/StreetViewContainer.jsx	Registers Panoramax panel in provider-to-panel mapping.
web/client/plugins/StreetView/containers/PanoramaxViewPanel.jsx	New Redux-connected panel for Panoramax view.
web/client/plugins/StreetView/constants.js	Adds `PANORAMAX` provider and default API URL constant.
web/client/plugins/StreetView/components/PanoramaxView/PanoramaxView.jsx	New Panoramax viewer component integration.
web/client/plugins/StreetView/api/panoramax.js	New Panoramax `getLocation` implementation (feature pick + API fallback).
web/client/plugins/StreetView/api/index.js	Exposes Panoramax API module alongside other providers.
web/client/components/map/openlayers/plugins/TileProviderLayer.js	Adds MVT support for tileprovider layers (VectorTileLayer/Source) and styling.
web/client/components/map/openlayers/Map.jsx	Adds `RenderFeature` handling so MVT features can be picked/intersected.
package.json	Adds `@panoramax/web-viewer` dependency.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

web/client/components/map/openlayers/Map.jsx

+        const type = geomLike.getType?.();
+        const coords = geomLike.getFlatCoordinates?.();
+        if (!type || !coords) return null;
+
+        switch (type) {
+        case 'Point': return new Point(coords);
+        case 'MultiPoint': return new MultiPoint([coords]);
+        case 'LineString': return new LineString(coords);
+        case 'MultiLineString': return new MultiLineString([coords]);
+        case 'Polygon': return new Polygon(coords);
+        case 'MultiPolygon': return new MultiPolygon([coords]);
+        default: return null; // types not supported
+        }


web/client/components/map/openlayers/plugins/TileProviderLayer.js

+        if (isMVT) {
+            const source = new VectorTileSource({
+                format: new MVT({}),
+                url: options.url,


web/client/components/map/openlayers/plugins/TileProviderLayer.js

+        const isMVT = options.format === 'application/vnd.mapbox-vector-tile';
+        // specific case of mvt layers
+        if (isMVT) {
+            const source = new VectorTileSource({
+                format: new MVT({}),
+                url: options.url,
+                maxZoom: options.maximumLevel ?? 22,
+                minZoom: options.minimumLevel ?? 0
+            });
+
+            const layer = new VectorTileLayer({
+                msId: options.id,
+                source,
+                visible: options.visibility !== false,
+                zIndex: options.zIndex,
+                opacity: options.opacity,
+                declutter: options.declutter ?? true,
+                preload: options.preload ?? 0,
+                cacheSize: options.cacheSize ?? 256,
+                tilePixelRatio: options.tilePixelRatio ?? 1,
+                renderBuffer: options.renderBuffer ?? 100,
+                renderMode: options.renderMode ?? 'hybrid' // or vector
+            });


web/client/plugins/StreetView/StreetView.jsx

+ *           "PanoramaxApiURL": "https://api.panoramax.xyz/api"
+ *           "minimumLevel": 0,
+ *           "maximumLevel": 15
+ *        }
+ *    ```
+ *   - `providerSettings` (optional). The settings specific for the provider. It is an object with the following properties:
+ *      - `providerSettings.PanoramaxApiURL` (optional). The URL of the Panoramax API. Default: `https://api.panoramax.xyz/api`.
+ *      - `providerSettings.srs` (optional). Coordinate reference system code to use for the API. Default: `EPSG:4326`. Note that the SRS used here must be supported by the Panoramax API **and** defined in `localConfig.json` file, in `projectionDefs`. This param is not used yet, panoramax api is implemented to receive coordinates only in SRS EPSG:4326
+ *      - `providerSettings.minimumLevel` The minimum zoom level at which the provider can provide tiles. Default value is 0
+ *      - `providerSettings.maximumLevel` The maximul zoom level at which the provider can provide tiles Default value is 15 according to the openstreetmap and IGN Panoramax instances
 * Generally speaking, you should prefer general settings in `localConfig.json` over the plugin configuration, in order to reuse the same configuration for default viewer and all the contexts, automatically. This way you will not need to configure the `apiKey` in every context.


web/client/plugins/StreetView/components/PanoramaxView/PanoramaxView.jsx

+import '@photo-sphere-viewer/core/index.css';
+import '@photo-sphere-viewer/markers-plugin/index.css';
+import '@photo-sphere-viewer/virtual-tour-plugin/index.css';
+import Message from "../../../locale/Message";


tdipisa requested a review from allyoucanmap December 16, 2025 15:58

tdipisa assigned allyoucanmap Dec 16, 2025

tdipisa added External Contribution enhancement labels Dec 16, 2025

tdipisa added this to the 2026.01.00 milestone Dec 16, 2025

nrc97 force-pushed the groupe-open/panoramax-streetview-support branch from 341255c to 5eed6a3 Compare December 17, 2025 16:43

nrc97 added 2 commits December 17, 2025 18:07

Translate panoramax emptySelection message in French, English, Spanis…

6e7f4bf

…h, Italian and German

nrc97 force-pushed the groupe-open/panoramax-streetview-support branch from 5eed6a3 to 6e7f4bf Compare December 17, 2025 17:08

tdipisa requested changes Jan 13, 2026

View reviewed changes

Merge branch 'master' into groupe-open/panoramax-streetview-support

010bba7

Remove unused DEFAULT_SRS constant

148b967

Removed unused DEFAULT_SRS constant from panoramax.js to match the eslint configuration

Merge branch 'master' into groupe-open/panoramax-streetview-support

ce1aa36

cla-bot bot added the CLA Ready label Feb 5, 2026

tdipisa added the backport 2026.01.xx label Feb 24, 2026

tdipisa modified the milestones: 2026.01.00, 2026.02.00 Mar 9, 2026

tdipisa removed the backport 2026.01.xx label Mar 9, 2026

Merge branch 'master' into groupe-open/panoramax-streetview-support

9c5d0bf

Copilot AI review requested due to automatic review settings March 18, 2026 17:06

Copilot started reviewing on behalf of nrc97 March 18, 2026 17:07 View session

Copilot AI reviewed Mar 18, 2026

View reviewed changes

Conversation

nrc97 commented Dec 16, 2025

Description

Issue

Breaking change

Other useful information

Uh oh!

tdipisa commented Dec 16, 2025

Uh oh!

tdipisa commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Gaetanbrl commented Dec 18, 2025

Uh oh!

landryb commented Jan 6, 2026

Uh oh!

tdipisa commented Jan 7, 2026

Uh oh!

nrc97 commented Jan 13, 2026

Uh oh!

tdipisa commented Jan 13, 2026

Uh oh!

tdipisa left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cla-bot bot commented Jan 21, 2026

Uh oh!

tdipisa commented Jan 22, 2026

Uh oh!

cla-bot bot commented Jan 28, 2026

Uh oh!

tdipisa commented Feb 24, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

tdipisa commented Dec 17, 2025 •

edited

Loading

tdipisa left a comment •

edited

Loading