Skip to content

fix(platform): list apps across all App Registry statuses - #206

Merged
jpage-godaddy merged 2 commits into
godaddy:mainfrom
dsingh5-godaddy:fix/DEVX-837-platform-app-list-all-statuses
Aug 12, 2026
Merged

fix(platform): list apps across all App Registry statuses#206
jpage-godaddy merged 2 commits into
godaddy:mainfrom
dsingh5-godaddy:fix/DEVX-837-platform-app-list-all-statuses

Conversation

@dsingh5-godaddy

Copy link
Copy Markdown
Contributor

Summary

  • gddy platform app list now sends an explicit App Registry status.in for every ApplicationStatus (ACTIVE, ARCHIVED, BLOCKED, INACTIVE, VERIFYING).
  • Fixes ACTIVE-only results caused by the API defaulting when status is omitted.

Test plan

  • cargo test --lib application::commands::list application::client::tests::list_applications
  • cargo run -- platform app list --env dev returns non-ACTIVE apps when present
  • Confirm help text mentions listing across every App Registry status

Jira: DEVX-837

dsingh5-godaddy and others added 2 commits August 12, 2026 18:20
App Registry defaults applications list to ACTIVE-only when status is omitted. Pass an explicit status.in of every ApplicationStatus so `gddy platform app list` returns all visible apps by default.

DEVX-837

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
"List all GoDaddy developer-platform applications visible to \
the current account. Use `gddy platform app info --name <name>` \
to fetch full details for a single application.",
the current account, across every App Registry status. Use \

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Wouldn't "all" already imply it's across all statuses?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

"all" isn’t a valid status — the enum is only ACTIVE | ARCHIVED | BLOCKED | INACTIVE | VERIFYING.

Omitting status also isn’t “all”: the API defaults to ACTIVE-only.

The CLI has to send status.in with every enum value, which is why the help text calls that out.

@jpage-godaddy
jpage-godaddy merged commit 737e91b into godaddy:main Aug 12, 2026
4 checks passed
This was referenced Aug 12, 2026
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