Skip to content

feat(weather): add native iOS widget weather refresh - #580

Merged
whes1015 merged 18 commits into
mainfrom
feat/ios-widget-weather-refresh
Oct 7, 2026
Merged

whes1015 merged 18 commits into
mainfrom
feat/ios-widget-weather-refresh

Conversation

@YinCheng0106

@YinCheng0106 YinCheng0106 commented Sep 20, 2026 •

Copy link
Copy Markdown
Member

這個 PR 做了什麼

讓 iOS 天氣小工具可在 App 未執行時,依照「所在地」或已儲存地點自行更新目前天氣,並在更新失敗時保留既有快取;使用「所在地」的 Widget 會顯示定位圖示以區分固定地點。

相關 issue

  • 無

怎麼驗

  • 在 iOS Simulator 加入天氣小工具,分別測試「所在地」與已儲存地點。
  • 設定為「所在地」時,確認地區名稱旁顯示定位圖示;已儲存的固定地點不顯示。
  • 關閉 DPIP App 後,確認 Widget Extension 仍可自行取得天氣並更新 snapshot。
  • 將 Simulator 所在地由臺中切換至臺北,確認 Current Location 可由 407 更新為 110。
  • 確認更新失敗時仍保留既有 snapshot。
  • 相關 native XCTest、tool/check.sh 與 GitHub CI 皆已通過。
  • 實體裝置 E2E 因 Apple Developer Team provisioning 權限限制暫未驗證。

檢查清單

  • tool/check/commits.sh origin/main..HEAD 通過
    —— commit 訊息就是更新日誌,格式見 commit.md
  • 一個 commit 一件事(這條 gate 驗不了,靠自己和 review)
  • mise exec -- flutter analyze 與 mise exec -- flutter test 通過
  • 新的使用者可見字串都走 AppLocalizations,沒有寫死
  • 有 UI 變更的話:用的是 AppSpacing / AppRadius / AppMotion,
    深色模式看過,文字對比度可接受

@YinCheng0106 YinCheng0106 self-assigned this Sep 20, 2026
@YinCheng0106 YinCheng0106 added the ios Issues or pull requests related to the iOS platform. label Sep 20, 2026
@YinCheng0106
YinCheng0106 added this pull request to stack #581 September 20, 2026 16:07
@YinCheng0106 YinCheng0106 added this to the v26.3 milestone Sep 20, 2026
@codecov

codecov Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.47%. Comparing base (5f4d94c) to head (716231c).

☔ View full report in Codecov by Harness.
📢 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.

@YinCheng0106
YinCheng0106 marked this pull request as ready for review September 22, 2026 03:38
Copilot AI balanced review requested due to automatic review settings September 22, 2026 03:38

This comment was marked as low quality.

@lowrt lowrt modified the milestones: v26.3, v26.4 Sep 23, 2026
@YinCheng0106
YinCheng0106 force-pushed the feat/ios-widget-weather-refresh branch 3 times, most recently from 14cf532 to 3e6d94a Compare September 28, 2026 07:07
@YinCheng0106
YinCheng0106 force-pushed the feat/ios-widget-weather-refresh branch from 3e6d94a to c33eb0f Compare September 28, 2026 07:42
@whes1015
whes1015 force-pushed the feat/ios-widget-weather-refresh branch from c33eb0f to e77d913 Compare October 2, 2026 23:17
lowrt
lowrt previously approved these changes Oct 3, 2026

@lowrt lowrt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@YinCheng0106 YinCheng0106 reopened this Oct 6, 2026
@whes1015
whes1015 dismissed lowrt’s stale review October 7, 2026 18:32

The merge-base changed after approval.

@whes1015
whes1015 force-pushed the feat/ios-widget-weather-refresh branch from e77d913 to 6e8cf4d Compare October 7, 2026 18:32
Base automatically changed from feat/ios-widget-intent-configuration to main October 7, 2026 21:55
Platform: ios

Fix(zh-Hant): 修正 iOS 天氣小工具偶爾顯示先前所在地的問題
Fix(en-US): fix iOS weather widgets occasionally showing a previous location
Platform: ios

New(zh-Hant): 使用「所在地」的天氣小工具會顯示定位圖示
New(en-US): weather widgets using Current Location now show a location icon
@whes1015
whes1015 force-pushed the feat/ios-widget-weather-refresh branch from 6e8cf4d to 716231c Compare October 7, 2026 22:00
@whes1015
whes1015 merged commit e12349f into main Oct 7, 2026
@whes1015
whes1015 deleted the feat/ios-widget-weather-refresh branch October 7, 2026 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ios Issues or pull requests related to the iOS platform. size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants