Skip to content

Conversation

@jjoderis
Copy link
Contributor

Summary

Added an environment variable that controls if the form editor is available in the UI + some additional bugfixes.

Details

  • env variable PROCEED_PUBLIC_PROCESS_AUTOMATION_TASK_EDITOR_ACTIVE added
    • depends on PROCEED_PUBLIC_PROCESS_AUTOMATION_ACTIVE to be true, otherwise the server will throw an error on startup
    • if not set to true the form editor will not appear in the sidebar and navigating to the route will show an 404 error
  • Fixed: The form editor is in a permanent read-only state
  • Fixed: The server console shows an error when opening the form editor due to a browser api being used in code that is prerendered on the server

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a specific requirement that the default value of PROCEED_PUBLIC_PROCESS_AUTOMATION_TASK_EDITOR_ACTIVE must be false? Otherwise, I think it should be true.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have settled on having "false" be the go to default value for these env variables.

Copy link

@Zayn-Javed Zayn-Javed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please look into the comment under src/management-system-v2/lib/ms-config/config-schema.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants