Skip to content

fix: UTCtoJST - #436

Merged
taminororo merged 1 commit into
developfrom
fix/yamal/320/UTCtoJST
Jul 23, 2026
Merged

fix: UTCtoJST#436
taminororo merged 1 commit into
developfrom
fix/yamal/320/UTCtoJST

Conversation

@yamato0308

@yamato0308 yamato0308 commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

対応Issue

resolve #320

概要

DS部門からの要望に基づき,スプレッドシート(GAS)向けに返却しているシフトレビューの日時(作成日時・更新日時)がUTC表示(JSTより9時間遅い)になっていた問題を,日本時間(JST: Asia/Tokyo)に変換して出力するように修正した変更です.

画面スクリーンショット等

  • URL
    スクリーンショット

テスト項目

備考

Summary by CodeRabbit

  • 改善
    • レビューの作成日時・更新日時を、日本時間(Asia/Tokyo)で表示するようになりました。
    • 表示形式を「YYYY/MM/DD HH:MM:SS」に統一しました。
    • 日本時間の取得に失敗した場合も、従来の日時表示で継続して確認できます。

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 72b2566f-6f19-49e8-9362-f9268e39e23e

📥 Commits

Reviewing files that changed from the base of the PR and between 0c0d9c1 and 6b86074.

📒 Files selected for processing (1)
  • api/lib/usecase/review_usecase.go

📝 Walkthrough

Walkthrough

レビュー一覧とID指定取得におけるcreated/updatedの表示を、Asia/Tokyoへ変換した日時へ変更しました。タイムゾーン読み込み失敗時は従来のフォーマットを使用します。

Changes

レビュー時刻表示

Layer / File(s) Summary
レビュー日時のJST変換
api/lib/usecase/review_usecase.go
GetReviewsGASGetReviewGASByIDAsia/Tokyoを読み込み、成功時はJSTへ変換して日時を整形し、読み込み失敗時は従来の日時を整形します。

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed UTCからJSTへの日時変換修正を指しており、変更内容の主旨と一致しています。
Description check ✅ Passed 対応Issue、概要、スクリーンショット欄、テスト項目があり、テンプレート要件は概ね満たしています。
Linked Issues check ✅ Passed Issue #320のJST変更要求に対し、GAS返却日時をAsia/Tokyoへ変換する実装で要件を満たしています。
Out of Scope Changes check ✅ Passed 変更はレビュー日時のJST化に限定されており、明らかな対象外の追加変更は見当たりません。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/yamal/320/UTCtoJST

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@yamato0308
yamato0308 requested a review from taminororo July 23, 2026 11:11

@taminororo taminororo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTMです!

@taminororo
taminororo merged commit 19ffa76 into develop Jul 23, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

シフトの時刻をJSTに変更する

2 participants