Skip to content

Use React Hooks for Timer#2546

Merged
corinagum merged 7 commits intomicrosoft:masterfrom
spyip:feat-react-hook-timer
Nov 21, 2019
Merged

Use React Hooks for Timer#2546
corinagum merged 7 commits intomicrosoft:masterfrom
spyip:feat-react-hook-timer

Conversation

@spyip
Copy link
Copy Markdown
Contributor

@spyip spyip commented Nov 3, 2019

Fixes #2331.

Changelog Entry

Added

  • component: Fixes #2331. Updated timer to use React Hooks, by @spyip in PR #2546

Description

This PR is based on extracted code from PR #2364

Specific Changes

  • Updated component/src/Utils/Timer.js to use React Hooks

  • Testing Added
    -Existing test included

(Edited by @compulim: updated issue number and re-copied changelog from files)

@coveralls
Copy link
Copy Markdown

coveralls commented Nov 3, 2019

Coverage Status

Coverage increased (+0.009%) to 64.048% when pulling e80339d on spyip:feat-react-hook-timer into c78f82f on microsoft:master.

@corinagum
Copy link
Copy Markdown
Contributor

Please add the following to the 'fixes' line and the changelog: #2313

Thank you!

@corinagum corinagum merged commit a2bb227 into microsoft:master Nov 21, 2019
@corinagum corinagum added the R7 label Dec 3, 2019
@compulim compulim mentioned this pull request Dec 10, 2019
73 tasks
@corinagum corinagum mentioned this pull request Dec 13, 2019
73 tasks
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.

3 participants