Closed arturboronat closed 10 months ago
Thank you for these Artur I will look into them. The metro4 import issue points to something with build configuration as you shouldn't have to directly reference a full node_module
path.
Thanks. I'm using the configuration in https://github.com/yamtl/educationplatform/tree/yamtl-playground-08112023 which comes from the original repo. Not sure where the problem may be.
@barnettwilliam, can we discuss this when we meet this week?
Thank you for these Artur, I have opened issue #108 for the metro import issue and moved url encoding and puml changes to pull request #109 therefore closing this pull request.
Context
To facilitate the publication of the bundled platform as a static resource akin to the approach used in the Epsilon Playground, I have introduced several changes. These modifications enable the platform to run without the necessity of Docker.
The objective is to incorporate these updates in a way that retains the functionality of the existing Docker deployment.
Proposed Changes
import metro4
statement does not function in this static context - at least for me.:
and/
are replaced with their percent-encoding. While the platform works, the url cannot copy/pasted directly anymore.newPanelId
within theOutputConsole
, where it erroneously references the panel's ID instead of the expectedPanelDef
ID.outputType
currently set as a constant must be dynamically sourced from theaction
object'soutputType
property to ensure the correct rendering of PlantUML diagrams.