Skip to content
This repository was archived by the owner on Mar 12, 2026. It is now read-only.

Commit 5b34a45

Browse files
authored
Merge pull request #142 from nnemirovsky/fix/onhold-cards-url-path
fix: on-hold cards not showing in card table view
2 parents 79fb76b + ac67673 commit 5b34a45

1 file changed

Lines changed: 4 additions & 6 deletions

File tree

internal/api/card.go

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import (
44
"context"
55
"encoding/json"
66
"fmt"
7-
"net/url"
87
"time"
98
)
109

@@ -247,15 +246,14 @@ func (c *Client) GetOnHoldCardsInColumn(ctx context.Context, onHoldCardsURL stri
247246
return nil, nil
248247
}
249248

250-
// Extract path from full URL
251-
parsed, err := url.Parse(onHoldCardsURL)
252-
if err != nil {
253-
return nil, fmt.Errorf("failed to parse on-hold cards URL: %w", err)
249+
path := extractPathFromURL(onHoldCardsURL)
250+
if path == "" {
251+
return nil, fmt.Errorf("failed to extract path from on-hold cards URL: %s", onHoldCardsURL)
254252
}
255253

256254
var cards []Card
257255
pr := NewPaginatedRequest(c)
258-
if err := pr.GetAll(parsed.Path, &cards); err != nil {
256+
if err := pr.GetAll(path, &cards); err != nil {
259257
return nil, fmt.Errorf("failed to fetch on-hold cards: %w", err)
260258
}
261259
return cards, nil

0 commit comments

Comments
 (0)