Initial implementation

Author: cb341

app/views/timer_sessions/_timer_container.html.erb

@@ -1,4 +1,7 @@
-<div class="box" data-controller="form timer" data-timer-timezone-value="<%= offset_for_time_zone %>">
+<div class="box" data-controller="form timer" data-timer-timezone-value="<%= offset_for_time_zone %>"
+     data-form-share-copied-value="<%= t('timer_sessions.timer.share_copied') %>"
+     data-form-share-ignored-value="<%= t('timer_sessions.timer.share_ignored') %>"
+     data-form-session-active-value="<%= timer_session.persisted? %>">
   <div class="timer-container">
     <% active_timer_session = timer_session %>
     <%= labelled_form_for(active_timer_session,
@@ -88,6 +91,10 @@
           <%= t('timer_sessions.timer.continue_last_session') %>
           <%= sprite_icon('add') %>
         <% end %>
+        <button type="button" data-name="timer-share" data-action="click->form#share" class="ml-3">
+          <%= t('timer_sessions.timer.share') %>
+          <%= sprite_icon('link') %>
+        </button>
       <% end %>
     <% end %>
 

assets.src/src/redmine-tracky/controllers/form-controller.ts

@@ -9,6 +9,9 @@ export default class extends Controller {
   declare readonly absolutInputTarget: HTMLInputElement
   declare readonly descriptionTarget: HTMLInputElement
   declare readonly issueTargets: Element[]
+  declare readonly shareCopiedValue: string
+  declare readonly shareIgnoredValue: string
+  declare readonly sessionActiveValue: boolean
 
   private connected = false
 
@@ -21,8 +24,15 @@ export default class extends Controller {
     'absolutInput',
   ]
 
+  static values = {
+    shareCopied: String,
+    shareIgnored: String,
+    sessionActive: Boolean,
+  }
+
   public connect() {
     this.connected = true
+    this.showShareIgnoredNotice()
   }
 
   public disconnect() {
@@ -67,6 +77,78 @@ export default class extends Controller {
     this.dispatchUpdate(form)
   }
 
+  public share(_event: Event) {
+    const params = new URLSearchParams()
+    const issueIds = this.extractIssueIds()
+    const comments = this.descriptionTarget.value
+    const timerStart = this.startTarget.value
+    const timerEnd = this.endTarget.value
+
+    issueIds.forEach((id) => params.append('issue_ids[]', id))
+    if (comments) params.set('comments', comments)
+    if (timerStart) params.set('timer_start', timerStart)
+    if (timerEnd) params.set('timer_end', timerEnd)
+
+    const url = `${window.location.origin}${window.location.pathname}?${params.toString()}`
+
+    this.copyToClipboard(url).then(() => {
+      this.showFlashNotice(this.shareCopiedValue)
+    })
+  }
+
+  private copyToClipboard(text: string): Promise<void> {
+    if (navigator.clipboard?.writeText) {
+      return navigator.clipboard.writeText(text).catch(() => {
+        this.copyToClipboardFallback(text)
+      })
+    }
+    this.copyToClipboardFallback(text)
+    return Promise.resolve()
+  }
+
+  private copyToClipboardFallback(text: string) {
+    const textarea = document.createElement('textarea')
+    textarea.value = text
+    textarea.style.position = 'fixed'
+    textarea.style.opacity = '0'
+    document.body.appendChild(textarea)
+    textarea.select()
+    document.execCommand('copy')
+    document.body.removeChild(textarea)
+  }
+
+  private showShareIgnoredNotice() {
+    if (!this.sessionActiveValue) return
+
+    const urlParams = new URLSearchParams(window.location.search)
+    const hasShareParams = urlParams.has('comments') ||
+      urlParams.has('timer_start') ||
+      urlParams.has('timer_end') ||
+      urlParams.getAll('issue_ids[]').some((v) => v !== '')
+
+    if (hasShareParams) {
+      this.showFlashNotice(this.shareIgnoredValue)
+    }
+  }
+
+  private showFlashNotice(message: string) {
+    const flash = document.getElementById('flash_notice')
+    if (flash) {
+      flash.textContent = message
+      flash.style.display = ''
+      return
+    }
+
+    const container = document.getElementById('content')
+    if (!container) return
+
+    const notice = document.createElement('div')
+    notice.id = 'flash_notice'
+    notice.className = 'flash notice'
+    notice.textContent = message
+    container.prepend(notice)
+  }
+
   private extractIssueIds(): string[] {
     return (
       this.issueTargets

assets.src/src/redmine-tracky/controllers/issue-completion-controller.ts

@@ -18,7 +18,7 @@ export default class extends Controller {
 
   connect() {
     this.listenForInput()
-    this.fetchIssuesFromURL()
+    this.prefillFromURL()
   }
 
   private listenForInput() {
@@ -43,8 +43,16 @@ export default class extends Controller {
     )
   }
 
-  private fetchIssuesFromURL() {
+  private prefillFromURL() {
     const urlParams = new URLSearchParams(window.location.search)
+
+    this.prefillIssuesFromURL(urlParams)
+    this.prefillFieldFromURL(urlParams, 'comments', '#timer_session_comments')
+    this.prefillFieldFromURL(urlParams, 'timer_start', '#timer_session_timer_start')
+    this.prefillFieldFromURL(urlParams, 'timer_end', '#timer_session_timer_end')
+  }
+
+  private prefillIssuesFromURL(urlParams: URLSearchParams) {
     const issueIds = urlParams.getAll('issue_ids[]')
 
     issueIds.filter(v => v !== "").forEach((id) => {
@@ -62,6 +70,17 @@ export default class extends Controller {
     })
   }
 
+  private prefillFieldFromURL(urlParams: URLSearchParams, param: string, selector: string) {
+    const value = urlParams.get(param)
+    if (!value) return
+
+    const field = document.querySelector<HTMLInputElement>(selector)
+    if (field) {
+      field.value = value
+      field.dispatchEvent(new Event('change'))
+    }
+  }
+
   private addIssue(issue: { item: CompletionResult }) {
     const listController =
       this.application.getControllerForElementAndIdentifier(

assets/javascripts/redmine-tracky.js

@@ -73,6 +73,9 @@ de:
     timer:
       start: Start
       continue_last_session: Anschliessend Starten
+      share: Teilen
+      share_copied: Link in die Zwischenablage kopiert
+      share_ignored: Ein Timer läuft bereits. Die geteilten Parameter wurden ignoriert.
       stop: Stop
       cancel: Abbrechen
       date_placeholder: 'dd.mm.yyyy hh:mm'

config/locales/de.yml

@@ -77,6 +77,9 @@ en:
     timer:
       start: Start
       continue_last_session: Start on end of last session
+      share: Share
+      share_copied: Link copied to clipboard
+      share_ignored: A timer is already running. Shared session parameters were ignored.
       stop: Stop
       cancel: Cancel
       date_placeholder: 'dd.mm.yyyy hh:mm'

config/locales/en.yml

@@ -119,6 +119,62 @@ class TimerManagementTest < ApplicationSystemTestCase
     assert_no_text Issue.second.subject
   end
 
+  test 'loading timer with comments from url' do
+    visit timer_sessions_path(comments: 'Meeting with team')
+
+    assert_equal 'Meeting with team', find('#timer_session_comments').value
+  end
+
+  test 'loading timer with timer_start from url' do
+    visit timer_sessions_path(timer_start: '01.01.2026 09:00')
+
+    assert_equal '01.01.2026 09:00', find('#timer_session_timer_start').value
+  end
+
+  test 'loading timer with timer_end from url' do
+    visit timer_sessions_path(timer_end: '01.01.2026 17:00')
+
+    assert_equal '01.01.2026 17:00', find('#timer_session_timer_end').value
+  end
+
+  test 'loading timer with all query params from url' do
+    visit timer_sessions_path(
+      issue_ids: [Issue.first.id],
+      comments: 'Sprint planning',
+      timer_start: '01.01.2026 09:00',
+      timer_end: '01.01.2026 10:00'
+    )
+
+    assert_text Issue.first.subject
+    assert_equal 'Sprint planning', find('#timer_session_comments').value
+    assert_equal '01.01.2026 09:00', find('#timer_session_timer_start').value
+    assert_equal '01.01.2026 10:00', find('#timer_session_timer_end').value
+  end
+
+  test 'share button is visible and clickable' do
+    visit timer_sessions_path
+
+    fill_in 'timer_session_comments', with: 'Pairing session'
+
+    find('[data-name="timer-share"]', wait: 5).click
+    assert_text I18n.t('timer_sessions.timer.share_copied')
+  end
+
+  test 'share button not visible when timer is active' do
+    FactoryBot.create(:timer_session, finished: false, user: User.current)
+    visit timer_sessions_path
+
+    assert has_content?(I18n.t('timer_sessions.timer.stop'))
+    assert_no_selector('[data-name="timer-share"]')
+  end
+
+  test 'shows prefill notice when active session exists and url has params' do
+    FactoryBot.create(:timer_session, finished: false, user: User.current)
+    visit timer_sessions_path(comments: 'Meeting', issue_ids: [Issue.first.id])
+
+    assert_text I18n.t('timer_sessions.timer.share_ignored')
+  end
+
   test 'preserves filter parameters when stopping a timer' do
     filter_date = 1.week.ago.strftime('%Y-%m-%d')
     current_date = Date.today.strftime('%Y-%m-%d')