Repository navigation
Bring the user documents in line with the code - #443
Merged
Merged
Conversation
A rejected API key or Authorization header answers HTTP 200 with -32600, not 401; Blazor WASM sends Encoded payloads, not encrypted ones; the UI package lists include Fluent and the WebAssembly host; removed samples, CSS classes and members are no longer named; the FieldValueChange method, the plugin binding check, the missing st_api_key table, the client-side error rebuild and BlazorUserContext's package are described as they are; the 1.13.0 notes say the JSON body codec now carries DateTime as a UTC instant.
…ass mapping Review follow-up: the Avalonia browser head sends Encoded payloads like Blazor WASM; bee-dynamic-form and bee-dynamic-grid map to polhem-form-layout and polhem-grid; the HTTP status statement is the default unless the host sets StatusCodeSelector, and a notification-only message gets 204.
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.
Part of #441
Documentation only (
docs/enand itsdocs/zh-TWtranslation, restamped). Each statement was checked against the code; an adversarial review then compared every changed sentence with the source.Authorizationheader answers HTTP 200 with-32600, not 401 (frontend integration, its TypeScript wrapper comment, cookbook, platform support). By default only a non-JSON body (415), a too-large body (413) and a notification-only message (204) get another status; a host can change that withJsonRpcHttpOptions.StatusCodeSelector, which Polhem does not set.Encodedpayloads, not encrypted ones; Blazor Server can be Local or Remote.AddPolhemBlazoris defined by each host.Staffform andDEMOcompany,polhem-dynamic-form/-grid(nowpolhem-form-layout/polhem-grid),DynamicGrid.EmptyText,ApiConnector.FinalizeResponse.FieldValueChangeamong the Plain-callable Form methods;BusinessResultResponsePairingTestscovering the framework's own result types; the plugin binding check (exactly the declared stage); a missingst_api_keytable letting calls through;BlazorUserContextinPolhem.Web.Blazor, used by both hosts;FormLiveComputationin both UI layers;IDbAccessFactorynever for company tables; the CI AOT gate's project list now points tobuild-ci.ymlinstead of copying it.DateTimeas a UTC instant.The dependency map items (B-07 to B-09) are left for the dependency map batch, which also makes
check-dependency-map.shcheck its package table.Verified locally:
check-docs-i18n.sh,check-public-docs.sh,check-md-links.sh,check-xmldoc-refs.sh,check-dependency-map.sh. Lite CI.