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

Stories #13

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Stories #13

wants to merge 5 commits into from

Conversation

carlosgcampos
Copy link

These are several commits to improve the report when using stories. It adds the duration of every story in the report and adds a new line after every story. It also adds the total hours per story in the aggregate report.

It's shown as (HH:MM story) making it possible to know the duration of
each task done in a particular day.
It makes easier to read for days having several tasks.
It helps a lot to find mistakes.
@@ -340,8 +340,15 @@ class DetailedReport(Report):
self.formatter.generate_large_text("Onsite hours worked: %s" % DateUtils.format_delta(onsite_time))

def get_stories_for_day_and_user(self, user, date):
def story_text(task):
Copy link
Owner

Choose a reason for hiding this comment

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

Maybe if the task doesn't have a story it doesn't make sense to add the time either? Also, if there is only one task for the day, isn't adding the story time a little redundant?

@mrobinson
Copy link
Owner

This is great! I had a couple questions, but it is probably safe to merge this and then just adjust things gradually to make it less awkward with the various ways that people use PHPReport.

@philn
Copy link
Contributor

philn commented Aug 9, 2019

I would be interested to have this feature accepted ;)

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