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: adding insights #335

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from
Draft

feat: adding insights #335

wants to merge 7 commits into from

Conversation

adam-buckley
Copy link
Contributor

@adam-buckley adam-buckley commented Aug 14, 2024

Checklist

  • I'm using the correct PHP Version (8.1 for current, 7.4 for legacy).
  • I've added comments to any new methods I've created or where else relevant.

Description

Adds My Timelog and Task Timelog insights, fixes some issues

Changelog

  • add My Timelog insight
  • add Task Timelog insight
  • adjust colour of table rows to be clearer
  • fix issue where the filter form was executed twice
  • fix ordering of insights in primary list
  • removed some unused code in runInsight
  • correct PHPDocs for some Html class elements
  • corrected some code formatting in line with our PHP CS standards
  • added some type casting to Web::ctx and _db property

refs:
issues:

Other Information

Docs pull request:

@adam-buckley
Copy link
Contributor Author

@careck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant