Conversation
Enable the Active Goals toolbar's Print All button: it generates a single printable PDF of the selected cohort's goals (for NSO hard copies) and downloads it via the CSV-export anchor pattern, gated on the cohort's training costs being entered like Run & Send. The printCohortGoals mutation (MPDX-9691) doesn't exist yet — the whole MpdGoalAdmin tool is still a mock prototype — so the generate step builds a placeholder PDF client-side behind the same generate-then-download seam the mutation will slot into. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
Bundle sizes [mpdx-react]Compared against 8b6c94e No significant changes found |
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.
Description
exportRest.tsx): spinner while generating, temporary-anchor download, error snackbar on failure.printCohortGoalsmutation (MPDX-9691) does not exist yet (no mpdx_api PR, not on staging), and MpdGoalAdmin is still the mock prototype tool, sogenerateCohortGoalsPdfcurrently builds a valid placeholder PDF client-side behind the exact seam the mutation will slot into (markedTODO(MPDX-9691)). CI codegen against the real schema also forbids referencing the mutation today.Testing
MPD Goals - Fall NSO 2026.pdffile downloads and opens, listing the cohort's goals (placeholder content until MPDX-9691 is wired)Checklist:
/quality:agent-reviewcommand locally and fixed any relevant suggestions🤖 Generated with Claude Code