fix(workspace): retrieve complete subscribed exam lists - #68
Merged
Merged
Conversation
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
1 participant
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.
workspace exampreviously treated the overview's three sample exams as a complete list and reported the sample size as the total. It now reads the paginated subscribed-exam REST endpoint, retrieves every page by default, shows the true total and semester, and supports date/semester/unknown-date filters plus explicit single-page--page/--limitoptions. A failed later page returns an error before any partial success output.The CLI requests
workspace.exam:read; README explains that existing users must runlife-ustc account loginto grant it. OpenAPI and generated clients are synchronized with the merged server implementation in Life-USTC/server#1137 using its reachable source commit and recorded hash.Validation:
make build,make test(including race and contract-script tests),make lint,make vet, OpenAPI source/reachability checks, and generated-code consistency. HTTP command regressions cover 104 exams over two pages, explicit pagination, filter forwarding, real total/semester output, and second-page authorization failure. No dependency or compatibility fallback was added.