Skip to content

Conversation

@EmilianoSanchez
Copy link
Contributor

@EmilianoSanchezEmilianoSanchez commented Oct 8, 2024

JavaScript commons library

What did you accomplish?

  • Removed promise wrapper to fix an issue with the client ready method that caused the returned promise to hang on async/await syntax if it was rejected.
  • The fix implies a breaking change since now the user must handle the promise rejection explicitly.

How do we test the changes introduced in this PR?

  • Unit tests and E2E tests in JS SDK.

Extra Notes

@EmilianoSanchezEmilianoSanchez requested a review from a team as a code ownerOctober 8, 2024 15:41
@EmilianoSanchezEmilianoSanchez changed the base branch from breaking_changes to breaking_changes_remove_deprecated_featuresOctober 8, 2024 18:27
@EmilianoSanchezEmilianoSanchez changed the title [Breaking change] Fix client ready() method when using async/await syntax[Breaking change] Fix client ready() method when using async/await syntax [ON HOLD]Oct 8, 2024
Base automatically changed from breaking_changes_remove_deprecated_features to breaking_changes_update_supported_runtimesOctober 18, 2024 16:32
Base automatically changed from breaking_changes_update_supported_runtimes to breaking_changesOctober 18, 2024 16:32
Base automatically changed from breaking_changes to breaking_changes_baselineOctober 18, 2024 16:36
@EmilianoSanchezEmilianoSanchez marked this pull request as draft October 18, 2024 16:43
Base automatically changed from breaking_changes_baseline to SDKS-8407_baselineNovember 1, 2024 20:46
Base automatically changed from SDKS-8407_baseline to developmentNovember 2, 2024 00:30
@EmilianoSanchezEmilianoSanchez changed the base branch from development to readiness-fix-ready-promiseOctober 22, 2025 21:10
@EmilianoSanchezEmilianoSanchez changed the title [Breaking change] Fix client ready() method when using async/await syntax [ON HOLD][Breaking change] Remove deprecated client ready() methodOct 22, 2025
@EmilianoSanchezEmilianoSanchez changed the base branch from readiness-fix-ready-promise to developmentOctober 22, 2025 21:18
@EmilianoSanchezEmilianoSanchez changed the base branch from development to readiness-fix-ready-promiseOctober 22, 2025 21:18
Base automatically changed from readiness-fix-ready-promise to readiness-baselineOctober 27, 2025 21:38
Base automatically changed from readiness-baseline to developmentOctober 27, 2025 21:53
Sign up for freeto 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.

1 participant

@EmilianoSanchez