Skip to content

feat: implement compressed log sharing functionality [WPB-26564]#4863

Merged
MohamadJaara merged 4 commits into
developfrom
mo/feat/share-logs-as-one-zip-file
Jun 24, 2026
Merged

feat: implement compressed log sharing functionality [WPB-26564]#4863
MohamadJaara merged 4 commits into
developfrom
mo/feat/share-logs-as-one-zip-file

Conversation

@MohamadJaara

@MohamadJaara MohamadJaara commented May 22, 2026

Copy link
Copy Markdown
Member

https://wearezeta.atlassian.net/browse/WPB-26564

rn log files are shared indivdually
This PR group them all in one zip file that is easier to share

@codecov

codecov Bot commented May 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 20.45455% with 70 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.99%. Comparing base (b36ff2d) to head (7baeb81).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...kotlin/com/wire/android/util/logging/LogSharing.kt 22.78% 54 Missing and 7 partials ⚠️
.../com/wire/android/ui/home/settings/SettingsItem.kt 0.00% 6 Missing ⚠️
...wire/android/ui/home/settings/SettingsViewModel.kt 0.00% 3 Missing ⚠️

❌ Your patch check has failed because the patch coverage (20.45%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4863      +/-   ##
===========================================
- Coverage    49.09%   48.99%   -0.11%     
===========================================
  Files          646      647       +1     
  Lines        23272    23357      +85     
  Branches      3571     3584      +13     
===========================================
+ Hits         11426    11444      +18     
- Misses       10784    10844      +60     
- Partials      1062     1069       +7     
Files with missing lines Coverage Δ
...wire/android/ui/home/settings/SettingsViewModel.kt 0.00% <0.00%> (ø)
.../com/wire/android/ui/home/settings/SettingsItem.kt 0.00% <0.00%> (ø)
...kotlin/com/wire/android/util/logging/LogSharing.kt 22.78% <22.78%> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b36ff2d...7baeb81. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MohamadJaara MohamadJaara force-pushed the mo/feat/share-logs-as-one-zip-file branch from 48e33ed to 9846e21 Compare May 22, 2026 20:44
@pull-request-size pull-request-size Bot added size/M and removed size/S labels May 22, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@MohamadJaara MohamadJaara requested a review from Garzas June 24, 2026 11:42
@MohamadJaara MohamadJaara force-pushed the mo/feat/share-logs-as-one-zip-file branch from 9846e21 to f051666 Compare June 24, 2026 11:42
@MohamadJaara MohamadJaara changed the title feat: implement compressed log sharing functionality feat: implement compressed log sharing functionality [WPB-26564] Jun 24, 2026
@MohamadJaara MohamadJaara enabled auto-merge June 24, 2026 12:08
@MohamadJaara MohamadJaara added this pull request to the merge queue Jun 24, 2026
Merged via the queue into develop with commit 9f05a21 Jun 24, 2026
20 of 21 checks passed
@MohamadJaara MohamadJaara deleted the mo/feat/share-logs-as-one-zip-file branch June 24, 2026 12:42
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants