Skip to content

add request id to iframe requests#59

Merged
andrewkmin merged 3 commits intomainfrom
andrew/request-idempotency
Jan 23, 2025
Merged

add request id to iframe requests#59
andrewkmin merged 3 commits intomainfrom
andrew/request-idempotency

Conversation

@andrewkmin
Copy link
Contributor

@andrewkmin andrewkmin commented Jan 7, 2025

$title

SDK counterpart: tkhq/sdk#467

This mainly serves the case where there are concurrent, or serial, requests. Request IDs are used to disambiguate between multiple requests.

@andrewkmin andrewkmin force-pushed the andrew/request-idempotency branch from fc38fa1 to 7042f2a Compare January 22, 2025 17:50
@andrewkmin andrewkmin force-pushed the andrew/request-idempotency branch from 442696a to c43ab1e Compare January 22, 2025 23:15
@andrewkmin andrewkmin requested review from r-n-o and turnekybc January 22, 2025 23:38
* @param {string} settings: JSON-stringified settings
* @param {string} requestId
*/
async function onApplySettings(settings) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see a requestId param added here

@andrewkmin andrewkmin merged commit 573a26d into main Jan 23, 2025
6 checks passed
@andrewkmin andrewkmin deleted the andrew/request-idempotency branch January 23, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants