-
Notifications
You must be signed in to change notification settings - Fork 299
feat: Support for CoreCrypto 9 [WPB-20699] #19608
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
Conversation
|
🔗 Download Full Report Artifact 🧪 Playwright Test Summary
Failed Tests:❌ Calls in channels with device switch and screenshare (tags: TC-8754, crit-flow-web)Location: specs/CriticalFlow/channelsCall-TC-8755.spec.ts:38 Errors: ❌ Channels Management (tags: TC-8752, crit-flow-web)Location: specs/CriticalFlow/channelsManagement-TC-8752.spec.ts:36 Errors: ❌ Planning group call with sending various messages during call (tags: TC-8632, crit-flow-web)Location: specs/CriticalFlow/groupCalls-TC-8632.spec.ts:37 Errors: ❌ Group Video call (tags: TC-8637, crit-flow-web)Location: specs/CriticalFlow/groupVideoCall-TC-8637.spec.ts:39 Errors: ❌ Messages in Channels (tags: TC-8753, crit-flow-web)Location: specs/CriticalFlow/messagesInChannels-TC-8753.spec.ts:44 Errors: ❌ Messages in Groups (tags: TC-8751, crit-flow-web)Location: specs/CriticalFlow/messagesInGroups-TC-8751.spec.ts:42 Errors: |
| await conversationService.joinByExternalCommit(qualifiedId); | ||
| await addOtherSelfClientsToMLSConversation(mlsConversation, selfUserQualifiedId, core.clientId, core); | ||
| // otherwise we should try to ensure the conversation exists (this will establish it if epoch is 0, or join by external commit if epoch > 0) | ||
| console.info('Conversation does not exist, ensuring establishment'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
console.info or logger.info?
…and clarify test description
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## dev #19608 +/- ##
==========================================
- Coverage 43.66% 43.64% -0.02%
==========================================
Files 1294 1294
Lines 32318 32343 +25
Branches 7177 7183 +6
==========================================
+ Hits 14111 14117 +6
- Misses 16515 16531 +16
- Partials 1692 1695 +3 🚀 New features to boost your workflow:
|
|



Description
In new core crypto version 9 the wasm file is moved to src/autogenerated and this PR changes the webpack config to copy all of src directory contents flattened into the dist of webapp