Conversation
✅ Deploy Preview for criipto-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
e002744 to
d855079
Compare
|
Looks like |
9d13a69 to
bcecc2d
Compare
| import '@fontsource/ibm-plex-sans/600.css'; | ||
| import '@fontsource/ibm-plex-sans/700.css'; | ||
| import '@fontsource/ibm-plex-mono/400.css'; | ||
| import 'graphiql/graphiql.min.css'; |
There was a problem hiding this comment.
Moved this import to global.css, otherwise the styles were not applied in the interactive tours.
| function base64ToArrayBuffer(base64: string): ArrayBuffer { | ||
| const binaryString = window.atob(base64); | ||
| const bytes = new Uint8Array(binaryString.length); | ||
| for (let i = 0; i < binaryString.length; i++){ | ||
| bytes[i] = binaryString.charCodeAt(i); | ||
| } | ||
| return bytes; | ||
| return bytes.buffer; | ||
| } |
There was a problem hiding this comment.
TypeScript inferred the old Uint8Array as an invalid BufferSource.
| isIduraBannerVisible: boolean; | ||
| onCloseBanner: () => void; | ||
| isIduraBannerVisible?: boolean; | ||
| onCloseBanner?: () => void; |
There was a problem hiding this comment.
A bit of an unrelated change, I know. But the two required props were causing a TS error in gatsby-browser.tsx which was changed in this PR, so I assumed it's acceptable. I'll move it to a separate commit.
There was a problem hiding this comment.
@nmoskaleva Of course acceptable, just good to document why, in a separate commit :)
|
Schema explorer looks & behaves as expected. |
Make isIduraBannerVisible and onCloseBanner optional to avoid TS errors when <DefaultLayout/> is used without the Idura banner.
bcecc2d to
532fd85
Compare
|
@mickhansen just a sec, rebase didn't go exactly as I was hoping, but I know what's wrong. |
| bytes[i] = binaryString.charCodeAt(i); | ||
| } | ||
| return bytes; | ||
| return bytes.buffer; |
There was a problem hiding this comment.
Not sure why this is required? subtle should support a typed array: https://developer.mozilla.org/en-US/docs/Web/API/SubtleCrypto/importKey
There was a problem hiding this comment.
I'm also not sure why, but after updating to React 19, there is a build error without this change @mickhansen
There was a problem hiding this comment.
@nmoskaleva What's the error? I don't see how React could influence the use of core browser APIs
There was a problem hiding this comment.
No overload matches this call. Overload 1 of 2, '(format: "jwk", keyData: JsonWebKey, algorithm: AlgorithmIdentifier | RsaHashedImportParams | EcKeyImportParams | HmacImportParams | AesKeyAlgorithm, extractable: boolean, keyUsages: readonly KeyUsage[]): Promise<...>', gave the following error. Argument of type '"raw"' is not assignable to parameter of type '"jwk"'. Overload 2 of 2, '(format: "raw" | "pkcs8" | "spki", keyData: BufferSource, algorithm: AlgorithmIdentifier | RsaHashedImportParams | EcKeyImportParams | HmacImportParams | AesKeyAlgorithm, extractable: boolean, keyUsages: KeyUsage[]): Promise<...>', gave the following error. Argument of type 'Uint8Array<ArrayBufferLike>' is not assignable to parameter of type 'BufferSource'. Type 'Uint8Array<ArrayBufferLike>' is not assignable to type 'ArrayBufferView<ArrayBuffer>'. Types of property 'buffer' are incompatible. Type 'ArrayBufferLike' is not assignable to type 'ArrayBuffer'. Type 'SharedArrayBuffer' is missing the following properties from type 'ArrayBuffer': resizable, resize, detached, transfer, transferToFixedLength
For some reason, the same error didn't appear in checks now. But when you noticed CI errors earlier today – it was due to the fact I lost this commit.
I see the error in my code editor now, and pretty sure it might appear again once other check errors are fixed... @mickhansen
7ddb521 to
1dbfc06
Compare
haysch
left a comment
There was a problem hiding this comment.
Overall docs look as expected.
Signatures explorer and schema still works as expected.
The available Signatures Interactive tour also looks and works as expected (an error in TSA server in test means I cannot complete the tour by closing the order).
1dbfc06 to
4457ff3
Compare
… to ArrayBuffer TypeScript inferred the old Uint8Array<ArrayBufferLike> as invalid BufferSource. Using ArrayBuffer fixes this.
4457ff3 to
432c263
Compare
QA: Please verify that everything looks as expected, specifically the styles in the Signatures interactive tours.