cal/major-domo-v2
1
0
Fork 0
Code Issues 6 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

Update CLI Tool / Team Roster Labels #59

New Issue
Closed
opened 2026-03-02 20:35:18 +00:00 by cal · 1 comment
cal commented 2026-03-02 20:35:18 +00:00
Owner
Copy Link

Update the Major Domo CLI tool to change the labels of team rosters. It still shows the old short IL and long IL nomenclature that was originally designed in the API response. We now use minor league and injured list as the labels for these fields. The short IL roster should be labeled as the injured list. The long IL roster from the API response is actually the minor league roster.

Update the Major Domo CLI tool to change the labels of team rosters. It still shows the old short IL and long IL nomenclature that was originally designed in the API response. We now use minor league and injured list as the labels for these fields. The short IL roster should be labeled as the injured list. The long IL roster from the API response is actually the minor league roster.
cal added the
ai-working
 label 2026-03-05 04:00:52 +00:00
cal added the
status/in-progress
 label 2026-03-05 04:03:57 +00:00
cal removed the
status/in-progress
 label 2026-03-05 04:05:22 +00:00
cal commented 2026-03-05 04:05:26 +00:00
Author
Owner
Copy Link

PR opened: #63

Fix approach:

  • Added section headers ("Active Roster", "Minor League", "Injured List") to the main /roster summary embed so each of the three roster sections is clearly labeled — previously the embed showed position counts without any section label
  • Fixed a backwards docstring in team_service.get_team_roster (shortil = Injured List, longil = Minor League)
PR opened: https://git.manticorum.com/cal/major-domo-v2/pulls/63 **Fix approach:** - Added section headers ("Active Roster", "Minor League", "Injured List") to the main `/roster` summary embed so each of the three roster sections is clearly labeled — previously the embed showed position counts without any section label - Fixed a backwards docstring in `team_service.get_team_roster` (`shortil` = Injured List, `longil` = Minor League)
cal added
status/pr-open
ai-pr-opened
 and removed
ai-working
 labels 2026-03-05 04:05:40 +00:00
cal closed this issue 2026-03-07 03:26:17 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
ai-changes-requested

   
ai-pr-opened

   
ai-reviewed

   
ai-reviewing

   
ai-working

   
in-next-release

Fix included in the next-release branch, pending merge to main

  
status/in-progress

Work is actively in progress

  
status/pr-open

A pull request has been opened for this issue

No Label
ai-changes-requested
ai-pr-opened
ai-reviewed
ai-reviewing
ai-working
in-next-release
status/in-progress
status/pr-open
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/major-domo-v2#59
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?