Iframe mode Parramatta is stuk

[lexvand]

When we made the multiple dashboards functionality for the training mode, we introduced a bug that broke the iframe mode – '?iframe=true' is interpreted as the name of a dashboard.

Same bug is also present in Lizard-tile-dashboard but of lower priority there as it doesn't have a real iframe mode.
____________________________________________________________________________________________
Solutions:

PR's:
https://github.com/nens/lizard-tile-dashboard/pull/57
https://github.com/nens/parramatta-dashboard/pull/147

[FRNT-208] created by remco.gerlich

[lexvand]

Remco Gerlich

Hi Remco,

Deze url opent wel het dashboard:
https://parramatta.lizard.net/floodsmart/trainingscenario1/?iframe=true
Misschien dat de javascript echter niet meer aan de url ziet dat het nu iframemode is.

Hoe kwam je hier achter dat het niet werkte?
Heb je misschien een voorbeeld url van waar het niet werkt?

groet Tom

by 5b3b278169812b2ef3f78d91

[lexvand]

Deze werkt niet: https://parramatta.lizard.net/floodsmart/?iframe=true

Ik zie nu dat de link die daadwerkelijk gebruikt wordt https://parramatta.lizard.net/floodsmart/full/5?iframe=true is. Dat werkt nog wel. Alleen de basis URL niet dus.

Maakt prio een stuk lager, maar het is wel een bug en de fix is heel simpel dus ik stel voor het wel even te doen.

by remco.gerlich

[lexvand]

Remco Gerlich
Hi Remco,
2 pr's in de description.
Zou je kunnen reviewen?

groet Tom

by 5b3b278169812b2ef3f78d91

[lexvand]

Remco Gerlich

Hi remco,
Ik heb je commentaar verwerkt.

groet Tom

by 5b3b278169812b2ef3f78d91

[lexvand]

Both merged.

by remco.gerlich