Skip to content

RS Post Settings: Use device timezone for date picker#22704

Open
nbradbury wants to merge 3 commits intotrunkfrom
issue/rspost-utc-date-picker
Open

RS Post Settings: Use device timezone for date picker#22704
nbradbury wants to merge 3 commits intotrunkfrom
issue/rspost-utc-date-picker

Conversation

@nbradbury
Copy link
Contributor

@nbradbury nbradbury commented Mar 18, 2026

Description

The RS post settings date/time picker was using UTC for displaying and selecting dates, causing users in non-UTC timezones to see times offset from their local time. This switches to device timezone to match the behavior of the old post settings date picker.

In order to avoid confusion regarding the timezone, I've also updated the displayed date/time to include the timezone, which is something the older picker didn't have.

Testing instructions

Date picker timezone:

  1. Set device timezone to something other than UTC (e.g., US Eastern)
  2. Open RS post settings for a published post
  3. Note the displayed publish date/time
  • Verify the date and time are shown in the device's local timezone
  1. Tap the date field, select a new date, then select a new time
  • Verify the date picker shows the correct local date initially
  • Verify the time picker shows the correct local hour/minute initially
  • Verify the updated date/time displayed in settings reflects the device timezone
  1. Save the post and reopen settings
  • Verify the saved date/time is displayed correctly in device timezone

nbradbury and others added 2 commits March 18, 2026 06:59
The date/time picker was using UTC for display and selection,
causing users to see times offset from their local timezone.
Switch to device timezone to match the old post settings behavior.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dangermattic
Copy link
Collaborator

dangermattic commented Mar 18, 2026

1 Warning
⚠️ PR is not assigned to a milestone.

Generated by 🚫 Danger

Show the device timezone abbreviation (e.g., EST, PST) after the
formatted date so users know which timezone the date refers to.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sonarqubecloud
Copy link

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress Android by scanning the QR code below to install the corresponding build.

App NameWordPress Android
Build TypeDebug
Versionpr22704-4ac2945
Build Number1487
Application IDorg.wordpress.android.prealpha
Commit4ac2945
Installation URL2jqutvtjrils0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack Android by scanning the QR code below to install the corresponding build.

App NameJetpack Android
Build TypeDebug
Versionpr22704-4ac2945
Build Number1487
Application IDcom.jetpack.android.prealpha
Commit4ac2945
Installation URL02agqns9p6hqo
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@codecov
Copy link

codecov bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 0% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.49%. Comparing base (cc26b8d) to head (4ac2945).
⚠️ Report is 1 commits behind head on trunk.

Files with missing lines Patch % Lines
...android/ui/postsrs/screens/PostRsSettingsScreen.kt 0.00% 11 Missing ⚠️
...ress/android/ui/postsrs/PostRsSettingsViewModel.kt 0.00% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            trunk   #22704      +/-   ##
==========================================
- Coverage   37.49%   37.49%   -0.01%     
==========================================
  Files        2286     2286              
  Lines      120484   120489       +5     
  Branches    16501    16501              
==========================================
  Hits        45173    45173              
- Misses      71637    71642       +5     
  Partials     3674     3674              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nbradbury nbradbury marked this pull request as ready for review March 18, 2026 12:15
@nbradbury nbradbury requested a review from adalpari March 18, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants