[PR #22] [MERGED] Beta tester output handlers #234

New issue

Closed

opened 2026-02-26 21:34:15 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 21:34:15 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/22
Author: @cursor[bot]
Created: 1/20/2026
Status: ✅ Merged
Merged: 1/20/2026
Merged by: @rudrankriyam

Base: feature/beta-management ← Head: cursor/beta-tester-output-handlers-9bca

📝 Commits (1)

a0c9b4a Fix missing table/markdown output handlers for beta tester delete/invite results

📊 Changes

2 files changed (+56 additions, -2 deletions)

View changed files

📝 cmd/beta.go (+2 -2)
📝 internal/asc/client.go (+54 -0)

📄 Description

Add table/markdown output handlers for beta tester delete/invite results to fix silent fallback to JSON.

Previously, the beta-testers remove and beta-testers invite commands would output JSON even when --output table or --output markdown was specified, because BetaTesterDeleteResult and BetaTesterInvitationResult were passed as values (not pointers) and lacked dedicated handlers in the PrintTable and PrintMarkdown functions. This PR updates the result creation to use pointers and adds the necessary type switch cases and helper functions for proper table and markdown formatting.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/rudrankriyam/App-Store-Connect-CLI/pull/22 **Author:** [@cursor[bot]](https://github.com/apps/cursor) **Created:** 1/20/2026 **Status:** ✅ Merged **Merged:** 1/20/2026 **Merged by:** [@rudrankriyam](https://github.com/rudrankriyam) **Base:** `feature/beta-management` ← **Head:** `cursor/beta-tester-output-handlers-9bca` --- ### 📝 Commits (1) - [`a0c9b4a`](https://github.com/rudrankriyam/App-Store-Connect-CLI/commit/a0c9b4aa529f124e21ef92df621455f27e251566) Fix missing table/markdown output handlers for beta tester delete/invite results ### 📊 Changes **2 files changed** (+56 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `cmd/beta.go` (+2 -2) 📝 `internal/asc/client.go` (+54 -0) </details> ### 📄 Description Add table/markdown output handlers for beta tester delete/invite results to fix silent fallback to JSON. Previously, the `beta-testers remove` and `beta-testers invite` commands would output JSON even when `--output table` or `--output markdown` was specified, because `BetaTesterDeleteResult` and `BetaTesterInvitationResult` were passed as values (not pointers) and lacked dedicated handlers in the `PrintTable` and `PrintMarkdown` functions. This PR updates the result creation to use pointers and adds the necessary type switch cases and helper functions for proper table and markdown formatting. --- --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>