Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix][WRS-1190] Don't execute SDK load event if document is not loaded #237

Conversation

psamusev
Copy link
Contributor

@psamusev psamusev commented Nov 25, 2024

This PR

Related tickets

Screenshots

@psamusev psamusev requested a review from a team as a code owner November 25, 2024 13:09
Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

github-actions bot commented Nov 25, 2024

Test Results

141 tests   139 ✅  1m 37s ⏱️
 31 suites    2 💤
  1 files      0 ❌

Results for commit a9840af.

♻️ This comment has been updated with latest results.

Copy link
Contributor

PR bundle size is 1.029 MB and 77.034 KB more than limit.

@psamusev psamusev added Ready for QA Defines that PR is ready for QA and removed Ready for QA Defines that PR is ready for QA labels Nov 25, 2024
@psamusev psamusev added Ready for QA Defines that PR is ready for QA and removed Ready for QA Defines that PR is ready for QA labels Nov 26, 2024
@psamusev psamusev added the Ready for QA Defines that PR is ready for QA label Dec 3, 2024
Copy link
Contributor

github-actions bot commented Dec 3, 2024

PR bundle size is 1.029 MB and 77.175 KB more than limit.

@psamusev psamusev added Ready for QA Defines that PR is ready for QA and removed Ready for QA Defines that PR is ready for QA labels Dec 3, 2024
@psamusev psamusev merged commit 0f23a01 into main Dec 3, 2024
7 checks passed
@psamusev psamusev deleted the bugfix/WRS-1190-Prevent-sdk.loadDocument-when-document-is-not-able-to-load branch December 3, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready for QA Defines that PR is ready for QA
Development

Successfully merging this pull request may close these issues.

2 participants