Conversation
…o device The registry's max device count fell back to 1 for the twelve catalog awards that name no attachment type. A second MILPAC row of such an award then raised the displayed count to 1 with the type still null, and the canvas requested attachments/null/1.png. The request returned 404 and logged "Error loading ribbon attachment" on every render. The ribbon still drew plain, so the uniform looked right. The fallback is now 0. The existing increment guard never passes for a device-less award, so the count stays 0 and the canvas requests nothing. The renderer and the valor path are unchanged. Adds one red-to-green case through GetCanvasObject: two rows of the Basic Assault Course Ribbon draw a plain ribbon. Closes #244 > *This was generated by AI*
SyniRon
enabled auto-merge (squash)
September 15, 2026 00:25
Vercin-G
approved these changes
Sep 18, 2026
Collaborator
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #244
What changed
AwardRegistry.getMaxAwardCountnow returns 0, not 1, for an award whose catalog entry names no attachment device. One line, plus a comment.One new test case in
awardNumerals.test.js: two rows of the Basic Assault Course Ribbon draw a plain ribbon. It runs through the existingGetCanvasObjectseam with a stubbedfetch, next to the Meritorious Service Medal cluster case.Why
Twelve catalog awards have no device. For those, the max count fell back to 1, so a second MILPAC row raised
ribbonDisplayedAttachmentCountto 1 whileribbonAttachmentTypestayednull. The canvas then requestedattachments/null/1.png, got a 404, and logged "Error loading ribbon attachment" on every render. The ribbon still drew plain, so the uniform looked right.With a max of 0, the existing increment guard never passes for a device-less award. The count stays 0 and the canvas requests nothing. The renderer, the valor path (
oakClustersValor/0.png) and the catalog are untouched.For the reviewer
main(1 !== 0) and green here. Reverting the one line reddens it again.getMaxAwardCountwas checked.MedalWithValoroverwrites the max to 14 aftersuper(). All unit citations carry a device. The numeral awards clamp withMath.minagainst a table value, not the fallback.npm testexits 0. Prettier passes.