Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

TimersManager: Removed interface of timers manager #2024

Merged
merged 7 commits into from
Sep 20, 2021

Conversation

iLLiCiTiT
Copy link
Member

Goal

  • not use interface in TimersManager module

Changes

  • TimersManager does not use interface to connect with other modules
  • module must have defined timers_manager_connector attribute which has at least 2 methods stop_timer and start_timer
  • ftrack and clockify modules were modified to have that attribute and methods

@iLLiCiTiT iLLiCiTiT self-assigned this Sep 15, 2021
@iLLiCiTiT iLLiCiTiT added the type: enhancement Enhancements to existing functionality label Sep 15, 2021
Copy link
Member

@antirotor antirotor left a comment

Choose a reason for hiding this comment

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

looks ok and it doesn't crash anything. No idea how to test it more :)

@iLLiCiTiT iLLiCiTiT merged commit 54a273e into develop Sep 20, 2021
@iLLiCiTiT iLLiCiTiT deleted the feature/timers_manager_wihtout_interface branch December 13, 2021 15:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants