Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Timer on retro board #191

Merged
merged 56 commits into from
Dec 15, 2023
Merged

Feature: Timer on retro board #191

merged 56 commits into from
Dec 15, 2023

Conversation

Nomandes
Copy link
Contributor

@Nomandes Nomandes commented Nov 23, 2023

Custom Timepicker Component

Issue: #54

Description

Implemented a timer feature for the retro board which is shared between the participants and can only be controlled by the moderator.

Acceptance criteria

  • Button Timer to open up a timer dialog is added to the action buttons
  • Timer dialog offers two inputs for seconds and minutes and the buttons to start, stop and pause the timer.
  • Timer dialog offers buttons to increment the timer by 1 minute, 3 minutes and 5 minutes for quick access
  • When the timer runs and the quick button, e.g. for 5 minutes is clicked, the timer running timer is incremented by 5 minutes
  • When the timer is not running and the quick button, e.g. for 5 minutes is clicked, the timer is incremented by 5 minutes but not started automatically.
  • When a timer runs, the button shows the remaining time instead of Timer
  • When a timer finishes the timer button shakes
  • When opening the timer dialog, the user will find the last configured timer there by default (must not be stored in local storage)
  • Only the moderator can start, stop and pause the timer.
  • Initial timer is 5 minutes

@Nomandes Nomandes added the feature New feature or request label Nov 23, 2023
@Nomandes Nomandes requested a review from NearW November 23, 2023 11:04
@Nomandes Nomandes self-assigned this Nov 23, 2023
Copy link
Collaborator

@NearW NearW left a comment

Choose a reason for hiding this comment

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

additionally: Commit messages should be written in present and not past tense.

feat: implemented custom timepicker component for the timer
=> feat: implement custom timepicker component for the timer

feat: added timer, timer button and dialog
=> feat: add timer, timer button and dialog

packages/frontend/src/common/components/TimePicker.tsx Outdated Show resolved Hide resolved
packages/frontend/src/common/components/TimePicker.tsx Outdated Show resolved Hide resolved
packages/frontend/src/common/components/TimePicker.tsx Outdated Show resolved Hide resolved
packages/frontend/src/common/components/TimePicker.tsx Outdated Show resolved Hide resolved
packages/frontend/src/common/components/TimePicker.tsx Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/context/RetroContext.tsx Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/reducers/retroReducer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/reducers/retroReducer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/reducers/retroReducer.ts Outdated Show resolved Hide resolved
packages/frontend/src/common/components/TimePicker.tsx Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimedEffect.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimedEffect.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
packages/frontend/src/retro/hooks/useTimer.ts Outdated Show resolved Hide resolved
@Nomandes Nomandes merged commit c96652c into master Dec 15, 2023
1 check passed
@Nomandes Nomandes deleted the feature/timeboxing branch December 15, 2023 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants