Skip to content

fix(cronjob): render container cron jobs in "cronjob get"#1982

Merged
martin-helmich merged 2 commits into
masterfrom
fix/cronjob-get-container-target
Jul 3, 2026
Merged

fix(cronjob): render container cron jobs in "cronjob get"#1982
martin-helmich merged 2 commits into
masterfrom
fix/cronjob-get-container-target

Conversation

@martin-helmich

Copy link
Copy Markdown
Member

Summary

Follow-up to #1981 (container cron job support in cronjob list).

A cron job may target an app installation OR a container (a service running in a stack). mw cronjob get previously always resolved the cron job's app installation (useAppInstallation(cronjob.appId)), which fails or renders misleading output for container cron jobs.

CronJobDetails now branches on the cron job's target:

  • Container (service) targets — show the Stack and Container instead of an App, and render the container command as the execution target.
  • App targets — rendered as before (via appInstallationId, falling back to the deprecated appId).

Because the app-installation lookup now lives in a dedicated subcomponent, it is only performed for app cron jobs.

🤖 Generated with Claude Code

martin-helmich and others added 2 commits July 3, 2026 10:26
A cron job may target an app installation OR a container (a service
running in a stack). "cronjob get" previously always resolved the
cron job's app installation, which fails or renders misleading output
for container cron jobs.

Branch on the cron job's target: for service (container) targets, show
the stack and container instead of the app, and render the container
command as the execution target. App cron jobs are rendered as before.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Render the app / stack / container as part of the EXECUTION TARGET
section instead of the primary cron job details, keeping all
"where and how is this executed" information in one place.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@martin-helmich martin-helmich merged commit 97f5475 into master Jul 3, 2026
11 checks passed
@martin-helmich martin-helmich deleted the fix/cronjob-get-container-target branch July 3, 2026 08:34
mittwald-machine added a commit that referenced this pull request Jul 3, 2026
## [1.20.1](v1.20.0...v1.20.1) (2026-07-03)

### Bug Fixes

* **cronjob:** include container (service) cronjobs in list ([#1981](#1981)) ([5c03091](5c03091)), closes [#1980](#1980)
* **cronjob:** render container cron jobs in "cronjob get" ([#1982](#1982)) ([97f5475](97f5475)), closes [#1981](#1981)
* **release:** do not specify model explicitly when announcing release ([b21740f](b21740f))
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.

1 participant