[Bug] Ignore unexpected query parameters rather than raising an excep…

…tion (#539)

Co-authored-by: Alexey Snigir <[email protected]>

.github/workflows/test-integration-vizro-core.yml

@@ -48,7 +48,7 @@ jobs:
       # If these are fixed then this whole step can be removed.
       - name: Install Chrome and chromedriver
         run: |
-          export chrome_version=119.0.6045.105
+          export chrome_version=125.0.6422.141
           sudo apt-get update || true
           sudo apt-get install libu2f-udev -y
           wget "https://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_$chrome_version-1_amd64.deb"

vizro-core/changelog.d/20240618_120925_antony.milne_strict_checks.md

@@ -0,0 +1,47 @@
+<!--
+A new scriv changelog fragment.
+
+Uncomment the section that is right (remove the HTML comment wrapper).
+-->
+
+<!--
+### Highlights ✨
+
+- A bullet item for the Highlights ✨ category with a link to the relevant PR at the end of your entry, e.g. Enable feature XXX ([#1](https://github.com/mckinsey/vizro/pull/1))
+
+-->
+<!--
+### Removed
+
+- A bullet item for the Removed category with a link to the relevant PR at the end of your entry, e.g. Enable feature XXX ([#1](https://github.com/mckinsey/vizro/pull/1))
+
+-->
+<!--
+### Added
+
+- A bullet item for the Added category with a link to the relevant PR at the end of your entry, e.g. Enable feature XXX ([#1](https://github.com/mckinsey/vizro/pull/1))
+
+-->
+<!--
+### Changed
+
+- A bullet item for the Changed category with a link to the relevant PR at the end of your entry, e.g. Enable feature XXX ([#1](https://github.com/mckinsey/vizro/pull/1))
+
+-->
+<!--
+### Deprecated
+
+- A bullet item for the Deprecated category with a link to the relevant PR at the end of your entry, e.g. Enable feature XXX ([#1](https://github.com/mckinsey/vizro/pull/1))
+
+-->
+
+### Fixed
+
+- Ignore unexpected query parameters rather than raising an exception ([#539](https://github.com/mckinsey/vizro/pull/539))
+
+<!--
+### Security
+
+- A bullet item for the Security category with a link to the relevant PR at the end of your entry, e.g. Enable feature XXX ([#1](https://github.com/mckinsey/vizro/pull/1))
+
+-->

vizro-core/src/vizro/models/_dashboard.py

@@ -246,7 +246,9 @@ def _arrange_page_divs(self, page_divs: _PageDivsType):
         page_main = html.Div(id="page-main", children=[collapsable_left_side, collapsable_icon, right_side])
         return html.Div(children=[page_header, page_main], className="page-container")
 
-    def _make_page_layout(self, page: Page):
+    def _make_page_layout(self, page: Page, **kwargs):
+        # **kwargs are not used but ensure that unexpected query parameters do not raise errors. See
+        # https://github.com/AnnMarieW/dash-multi-page-app-demos/#5-preventing-query-string-errors
         page_divs = self._get_page_divs(page=page)
         page_layout = self._arrange_page_divs(page_divs=page_divs)
         page_layout.id = page.id