Skip to content

feat: add task publish and unpublish commands - #1317

Open
Janjiran wants to merge 2 commits into
masterfrom
feat/task-publication
Open

feat: add task publish and unpublish commands#1317
Janjiran wants to merge 2 commits into
masterfrom
feat/task-publication

Conversation

@Janjiran

@Janjiran Janjiran commented Aug 3, 2026

Copy link
Copy Markdown

@Janjiran Janjiran added this to the 146th sprint - Growth team milestone Aug 3, 2026
@Janjiran Janjiran self-assigned this Aug 3, 2026
@Janjiran Janjiran added the t-growth Issues owned by the Growth team. label Aug 3, 2026
@Janjiran
Janjiran requested a review from katzino August 3, 2026 06:47
Comment thread docs/reference.md
Comment on lines +1622 to +1623
task publish Publishes the task on its public landing page.
task unpublish Unpublishes the task from its public landing

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.

Question: I understand those two commands but how can one create/update tasks via CLI so they actually have publicConfig set? publish commands already assumes it is set and valid, right?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

It's not possible today, but it's also not possible for the Actors, but I agree it's kind of strange.

Comment thread src/commands/task/publish.ts
Comment thread src/commands/task/publish.ts Outdated
Comment thread src/commands/task/publish.ts Outdated
Comment thread src/commands/task/publish.ts
@Janjiran
Janjiran requested a review from katzino August 6, 2026 13:22

@katzino katzino 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.

LGTM
One suggestion, pre-approved ✅

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.

Important: You should probably import process from 'node:process';

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.

Important: You should probably import process from 'node:process';

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

Labels

t-growth Issues owned by the Growth team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants