Timesheet component missing features

[mihaisolomon]

I think we are missing a few things from this component

• [x] Daily/weekly reports on the time logged - PR: https://github.com/devaslanphp/project-management/pull/53
• [x] Logged time split by activity - PR: https://github.com/devaslanphp/project-management/pull/44
• [x] Monthly reports on time logged
  • PR: https://github.com/devaslanphp/project-management/pull/38
  • PR: https://github.com/devaslanphp/project-management/pull/44
  • https://github.com/devaslanphp/project-management/pull/53
• [ ] Target hours per moth customised per user (maybe)
• [ ] Split hours between lets say billable and non billable
• [x] Timesheet listing - PR: https://github.com/devaslanphp/project-management/pull/33

If think of anything else please add.

[heloufir]

Hmm I see, do you think of a separated dashboard where the user can view / export data as tables / graphs or something else?

[mihaisolomon]

Yes, something like that, and a widget in the main one with some basic information like reported hours this month.

[heloufir]

Ok I see, so it will like a reporting dashboard giving the user some quick KPI and options to export and view data.

[heloufir]

And I think I should add a date to the logging time so we can log time based on a date.

E.g. we can log 8 hours yesterday and 4 hours today in the same day, now we are based on the system current date.

I think Jira are doing the same.

[mihaisolomon]

Timesheet listing:

Menu:

I will add the possibility to edit a timesheet entry. Just edit the hours, activity and comment.

[heloufir]

That looks good!

[heloufir]

@mihaisolomon just tested it with a big comment and it looks like below

I think it would be good to add some limit to the comment column like these:

Tables\Columns\TextColumn::make('comment')
    ->label(__('Comment'))
    ->limit(50)
    ->sortable()
    ->searchable(),

[mihaisolomon]

I'm working on a small widget ( for the timesheet), I will fix this part also. Thanks.

[mihaisolomon]

The first widget looks something like this:

[heloufir]

That looks nice, you are using Filament Widgets?

[mihaisolomon]

That looks nice, you are using Filament Widgets?

Yes.

[heloufir]

@mihaisolomon I will hide the Timesheet menu, so I can publish a new release containing the Scrum board without showing the Timesheet menu for now, that's ok for you?

[heloufir]

Just merged the PR #38 When I go to the Timesheet Dashboard, I got this error

I think the DATE_FORMAT you are using on your SQL Query

[mihaisolomon]

What mysql version you are using?

I had the same issues with mysql 6 a while ago. Now I have 8.

[heloufir]

I am using MariaDB version 10.4.22

[mihaisolomon]

@heloufir Take a look over the timesheet dashboard and see if the error persists. I have the same code running on a server on MariaDb and could not replicate it.

[heloufir]

Yep always the same issue, here is the shared issue: https://flareapp.io/share/Rmr2Qqkm#F108

[heloufir]

@mihaisolomon The issue is the space below 😱

You just need to remove the space in there, and the issue is gone.

[heloufir]

I got another issue related to my demo data, I don't really know what is the issue, I think I soft delete an activity somewhere But I think it will be good to check inside the foreach ($collection as $item) if the $item->activity != null before adding the assets, what do you think?

[heloufir]

@mihaisolomon Any news on this?

[mihaisolomon]

Yes, I will make an update later this week with a few more things, that contains the fix for it.

[mihaisolomon]

I will close this one because almost all of the components where done. Some of them they don't make sense for this. If you have any other ideas on this please make a new ticket.