Skip to content

Add D/DEVCOM and D/ACD to ADR and Roster Statistics#158

Merged
SyniRon merged 5 commits into
7Cav:mainfrom
HWqs:add-devcom-dacd
Jul 7, 2026
Merged

Add D/DEVCOM and D/ACD to ADR and Roster Statistics#158
SyniRon merged 5 commits into
7Cav:mainfrom
HWqs:add-devcom-dacd

Conversation

@HWqs

@HWqs HWqs commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Resolves 7cav.us ticket #12005 (D/DEVCOM missing from the ADR): https://7cav.us/tickets/12005/

What this does

  • ADR (adr/page.jsx): D/DEVCOM gets its own "Development Command" section. D/ACD goes under Auxiliary Combat Division.
  • Roster Statistics (BilletBank.jsx, rosterstatistics/page.jsx): adds D/ACD (delta4) and the DEVCOM arrays. Also refreshes charlie4 (C/ACD), which had drifted from what /roster/groups returns now.
  • Chart colors (statistics.jsx): the regiment chart assigns colors by position, so new segments need a matching entry or they show up blank. Added them for C/ACD and D/ACD.

Testing

Ran it locally: D/DEVCOM and D/ACD show the right members on the ADR, and the new segments render on Statistics. IDs pulled from /roster/groups.

Left out on purpose

@HWqs

HWqs commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Something messy happened with the README so doing that separately

Comment thread client/app/reusableModules/BilletBank.jsx
Comment thread client/app/rosterstatistics/page.jsx
Comment thread client/app/rosterstatistics/page.jsx
@SyniRon

SyniRon commented Jun 27, 2026

Copy link
Copy Markdown
Collaborator

Looks good overall, good work. Once those few issues are cleaned up we should be good to merge.

@HWqs HWqs requested a review from SyniRon July 2, 2026 22:43

@SyniRon SyniRon left a comment

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.

Looks good now ty

@SyniRon SyniRon merged commit 1d08e71 into 7Cav:main Jul 7, 2026
1 check passed
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.

2 participants