feat: add use case to check if backend supports personal to team migration (WPB-12022) #3114
Conversation
ohassine
requested review from
a team,
m-zagorski,
yamilmedina,
borichellow,
Garzas,
saleniuk and
damian-kaczmarek
and removed request for
a team
|
Bencher Report| Branch | add-use-case-to-check-if-backend-supports-personal-to-team-migration |
| Testbed | ubuntu-latest |
⚠️ WARNING: No Threshold found!Without a Threshold, no Alerts will ever be generated.
Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the--ci-only-thresholdsCLI flag.
Click to view all benchmark results
| Benchmark | Latency | nanoseconds (ns) |
|---|---|---|
| com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInFiles | 📈 view plot | 656,498.50 |
| com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInMemory | 📈 view plot | 389,933,952.42 |
| com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.messageInsertionBenchmark | 📈 view plot | 1,318,880,769.81 |
| com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.queryMessagesBenchmark | 📈 view plot | 21,506,707.53 |
Datadog ReportBranch report: ✅ 0 Failed, 3202 Passed, 107 Skipped, 34.41s Total Time |
...lium/logic/feature/personaltoteamaccount/IsPersonalToTeamAccountSupportedByBackendUseCase.kt
Outdated
Show resolved
Hide resolved
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3114 +/- ##
===========================================
- Coverage 54.03% 54.02% -0.01%
===========================================
Files 1246 1247 +1
Lines 36152 36176 +24
Branches 3656 3658 +2
===========================================
+ Hits 19534 19545 +11
- Misses 15208 15221 +13
Partials 1410 1410
... and 2 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
…eck-if-backend-supports-personal-to-team-migration
…-personal-to-team-migration
|
There was a problem hiding this comment.
this is not 100% guarantied to be correct at run time, there is a small chance that an API version is updated after the user session is created therefore the update will not take place until the next app restart
to be 100% accurate you can get the server config from SessionManager.serverConfig()
...n/com/wire/kalium/logic/feature/personaltoteamaccount/CanMigrateFromPersonalToTeamUseCase.kt
Outdated
Show resolved
Hide resolved
|
ohassine
deleted the
add-use-case-to-check-if-backend-supports-personal-to-team-migration
branch
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764The PR Description
What's new in this PR?
Issues
Use case to check if backend supports personal to team account migration
Needs releases with:
Testing
Test Coverage (Optional)
How to Test
Briefly describe how this change was tested and if applicable the exact steps taken to verify that it works as expected.
Notes (Optional)
Specify here any other facts that you think are important for this issue.
Attachments (Optional)
Attachments like images, videos, etc. (drag and drop in the text box)
PR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.