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

Rework estimated time #7

Closed
wants to merge 4 commits into from
Closed

Rework estimated time #7

wants to merge 4 commits into from

Conversation

davidbeig
Copy link
Member

This PR will do a rework so that there is an estimated time for ever activity and that it can be translated into accountability when exported.

@codecov
Copy link

codecov bot commented Sep 17, 2020

Codecov Report

Merging #7 into master will decrease coverage by 0.28%.
The diff coverage is 40.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   61.01%   60.73%   -0.29%     
==========================================
  Files          46       46              
  Lines         867      871       +4     
==========================================
  Hits          529      529              
- Misses        338      342       +4     
Impacted Files Coverage Δ
...ands/decidim/time_tracker/admin/create_activity.rb 92.30% <ø> (-7.70%) ⬇️
.../decidim/time_tracker/admin/export_time_tracker.rb 0.00% <0.00%> (ø)
...ands/decidim/time_tracker/admin/update_activity.rb 93.75% <ø> (-6.25%) ⬇️
.../forms/decidim/time_tracker/admin/activity_form.rb 100.00% <100.00%> (ø)
...ecidim/time_tracker/admin/activities_controller.rb 87.23% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97b10bb...50e0b87. Read the comment docs.

@microstudi microstudi marked this pull request as draft September 18, 2020 08:52
@microstudi microstudi added the wip label Sep 18, 2020
@microstudi microstudi assigned verarojman and unassigned microstudi Dec 9, 2020
@verarojman
Copy link
Contributor

Superseded by #15

@verarojman verarojman closed this Dec 9, 2020
@verarojman verarojman mentioned this pull request Dec 9, 2020
3 tasks
@verarojman verarojman added done duplicate This issue or pull request already exists and removed wip done labels Dec 10, 2020
@verarojman verarojman deleted the rework-estimated-time branch January 18, 2021 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants