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: Localised DateFormat in device info #2785

Merged

Conversation

AndroidBob
Copy link
Collaborator

Cherry pick from the original PR:


⚠️ Conflicts during cherry-pick:
kalium

What's new in this PR?

Issues

Devices screen - Time and date format not respecting locale

Causes (Optional)

Was not implemented

Solutions

implement it

Attachments (Optional)

<img width=378 alt=Screenshot 2024-03-11 at 23 32 11 src=https://github.com/wireapp/wire-android/assets/6539347/23e06ab6-df18-40f6-a965-6c4c299b958e>

@AndroidBob AndroidBob added the cherry-pick PR is cherry-picking changes from another banch label Mar 12, 2024
Copy link

codecov bot commented Mar 12, 2024

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 43.47%. Comparing base (724816c) to head (3758734).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2785      +/-   ##
===========================================
- Coverage    43.47%   43.47%   -0.01%     
===========================================
  Files          411      411              
  Lines        13867    13872       +5     
  Branches      2517     2518       +1     
===========================================
+ Hits          6029     6031       +2     
- Misses        7130     7132       +2     
- Partials       708      709       +1     
Files Coverage Δ
.../main/kotlin/com/wire/android/util/DateTimeUtil.kt 63.46% <60.00%> (-2.50%) ⬇️

Continue to review full report in Codecov by Sentry.

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

Copy link
Contributor

github-actions bot commented Mar 12, 2024

Test Results

858 tests  +1   858 ✅ +1   14m 7s ⏱️ +12s
116 suites ±0     0 💤 ±0 
116 files   ±0     0 ❌ ±0 

Results for commit 3758734. ± Comparison against base commit 724816c.

♻️ This comment has been updated with latest results.

Copy link
Contributor

APKs built during tests are available here. Scroll down to Artifacts!

@AndroidBob
Copy link
Collaborator Author

Build 3546 succeeded.

The build produced the following APK's:

@AndroidBob
Copy link
Collaborator Author

Build 3558 failed.

@AndroidBob
Copy link
Collaborator Author

Build 3566 failed.

…_format_in_device_info-cherry-pick

# Conflicts:
#	kalium
@ohassine ohassine enabled auto-merge March 12, 2024 19:55
@ohassine ohassine added this pull request to the merge queue Mar 12, 2024
Copy link
Contributor

APKs built during tests are available here. Scroll down to Artifacts!

Merged via the queue into develop with commit 04d6946 Mar 12, 2024
14 of 15 checks passed
@ohassine ohassine deleted the fix/localised_date_format_in_device_info-cherry-pick branch March 12, 2024 20:46
@AndroidBob
Copy link
Collaborator Author

Build 3571 succeeded.

The build produced the following APK's:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick PR is cherry-picking changes from another banch size/XS
Projects
None yet
4 participants