Skip to content

feat(web): add container name template - #1142

Merged
m8vago merged 2 commits into
developfrom
feat/add-container-name-template
Aug 20, 2026
Merged

m8vago merged 2 commits into
developfrom
feat/add-container-name-template

Conversation

@m8vago

@m8vago m8vago commented Aug 19, 2026 •

Copy link
Copy Markdown
Contributor
  • Add {{containerName}} template for environments in container config.
  • Fix public hub registry's 100 image limit

@m8vago
m8vago requested a review from a team as a code owner August 19, 2026 16:33
@github-actions github-actions Bot added pr:feat source:web The scope of the issue or pull request is web. lang:typescript and removed pr:feat labels Aug 19, 2026
@m8vago m8vago changed the title feat: add container name template feat(web): add container name template Aug 19, 2026
@m8vago
m8vago force-pushed the feat/add-container-name-template branch 2 times, most recently from f851c23 to a66660e Compare August 19, 2026 19:37
@m8vago
m8vago force-pushed the feat/add-container-name-template branch from a66660e to f77a9b3 Compare August 19, 2026 19:38

@nandor-magyar nandor-magyar left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What will happen after the first 100 **images? Are we showing anything?
LGTM anyway.

@m8vago

m8vago commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

What will happen after the first 100 **images? Are we showing anything? LGTM anyway.

We swallow the error, and list only the first 100 images.
There is also a warning in the registry edit screen.

@m8vago
m8vago merged commit 51b8853 into develop Aug 20, 2026
22 checks passed
@m8vago
m8vago deleted the feat/add-container-name-template branch August 20, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang:typescript source:web The scope of the issue or pull request is web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants