Skip to content

Conversation

@Hubert-Szczepanski-SAP
Copy link
Contributor

What this PR does / why we need it:

Packing UI into OCM + publish.

I Added local tests with local registry.

task local:test-full-flow: is to create OCM package and then use it to host app locally within Docker.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Dockerfile Outdated

# Build
ENV NODE_ENV=production
ENV NODE_OPTIONS=--max-old-space-size=4096
Copy link
Member

Choose a reason for hiding this comment

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

Is it without not working? Why we need to change the Dockerfile?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was not working for me when I was trying to deploy it locally

Copy link
Member

@n3rdc4ptn n3rdc4ptn left a comment

Choose a reason for hiding this comment

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

lgtm

@Hubert-Szczepanski-SAP Hubert-Szczepanski-SAP merged commit c6ecfcf into main Oct 23, 2025
5 checks passed
@Hubert-Szczepanski-SAP Hubert-Szczepanski-SAP deleted the feat/adding-ocm-package-for-new-version-release branch October 23, 2025 08:35
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