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

feat: add time management #547

Merged
merged 4 commits into from
Aug 3, 2022
Merged

Conversation

MikaelVallenet
Copy link
Collaborator

@MikaelVallenet MikaelVallenet commented Jul 27, 2022

The goal was to add a way to indicate the time required for a task from the github issue Resolve #545 .
For this i propose this feature which allows to indicate time in issues.

To use this feature you just need to include the keyword time followed by a space and the time value.

My feature takes into account different time units:

Colonne 1 Colonne 2
w weeks
d day
h hours
m minutes

Example of an issue with the indicated time:

issues

Then i add a hourglass component icon in JS and display after that the time

card

Here is before implementation

before

Here is after implementation

after

┆Issue is synchronized with this Trello card

@MikaelVallenet MikaelVallenet added the enhancement New feature or request label Jul 27, 2022
@MikaelVallenet MikaelVallenet self-assigned this Jul 27, 2022
@MikaelVallenet MikaelVallenet requested a review from moul as a code owner July 27, 2022 08:53
@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label Jul 27, 2022
api/dvmodel.proto Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #547 (d8e2137) into master (0fdc89f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #547   +/-   ##
=======================================
  Coverage   27.95%   27.95%           
=======================================
  Files           6        6           
  Lines         440      440           
=======================================
  Hits          123      123           
  Misses        315      315           
  Partials        2        2           
Flag Coverage Δ
unittests 27.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us.

@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review Pull Request is not reviewed yet labels Aug 3, 2022
@moul moul merged commit fe428f3 into moul:master Aug 3, 2022
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed ✨ Merged Pull Request has been merged successfully ✅ Approved Pull Request has been approved and can be merged labels Aug 3, 2022
@github-actions
Copy link

github-actions bot commented Aug 3, 2022

🎉 This PR is included in version 3.13.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Manage the time of an issue
2 participants