From 05d3375a7d26e622e7fc1cdb477cc9148cbd499b Mon Sep 17 00:00:00 2001 From: Jorge Manrubia Date: Thu, 11 Sep 2025 16:29:07 +0200 Subject: [PATCH] Avoid having to parse strings --- app/helpers/time_helper.rb | 2 +- app/javascript/controllers/local_time_controller.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/helpers/time_helper.rb b/app/helpers/time_helper.rb index 99ece604c..53fa055b5 100644 --- a/app/helpers/time_helper.rb +++ b/app/helpers/time_helper.rb @@ -1,6 +1,6 @@ module TimeHelper def local_datetime_tag(datetime, style: :time, **attributes) # Render empty space to ensure it takes height until the local time is loaded via JS - tag.time " ".html_safe, **attributes, datetime: datetime.iso8601, data: { local_time_target: style, action: "turbo:morph-element->local-time#refreshTarget" } + tag.time " ".html_safe, **attributes, datetime: datetime.to_i, data: { local_time_target: style, action: "turbo:morph-element->local-time#refreshTarget" } end end diff --git a/app/javascript/controllers/local_time_controller.js b/app/javascript/controllers/local_time_controller.js index 69693a107..40ff30f09 100644 --- a/app/javascript/controllers/local_time_controller.js +++ b/app/javascript/controllers/local_time_controller.js @@ -89,7 +89,7 @@ export default class extends Controller { } #formatTime(formatter, target) { - const dt = new Date(target.getAttribute("datetime")) + const dt = new Date(parseInt(target.getAttribute("datetime"))*1000) target.innerHTML = formatter.format(dt) target.title = this.datetimeFormatter.format(dt) }