Skip to content

Conversation

@StanBarrows
Copy link
Contributor

No description provided.

agross and others added 30 commits October 21, 2025 12:20
…ing in the rows

mounting the rows mounts lots of nested components which results in a delay on the initial mount.
Copilot AI review requested due to automatic review settings December 1, 2025 11:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a new organization-level permission setting employees_can_manage_tasks that allows employees to manage tasks in projects they have access to, while maintaining proper access controls for private vs public projects. The changes span across backend permission handling, frontend UI components, and various infrastructure improvements including shared UI theme extraction and improved time tracking features.

Key Changes:

  • Added employees_can_manage_tasks permission system for employee task management
  • Extracted shared Tailwind theme configuration to @solidtime/ui package for reuse
  • Enhanced time tracking UI with manual entry support and improved time range selectors
  • Added activity status visualization to calendar view with idle/active period tracking
  • Improved type safety and removed duplicate code across components

Reviewed changes

Copilot reviewed 82 out of 88 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tsconfig.json Added path alias for @solidtime/ui package
tests/Unit/Service/PermissionStoreTest.php Added comprehensive tests for new employees_can_manage_tasks permission system
tests/Unit/Endpoint/Api/V1/TaskEndpointTest.php Updated task permission tests to use :all permissions and added employee task management tests
tests/Unit/Endpoint/Api/V1/ClientEndpointTest.php Added clients:view:all permission and test for scoped client visibility
tests/Unit/Console/KernelTest.php New test file for console kernel schedule validation
tailwind.config.js Refactored to use shared theme from @solidtime/ui package
resources/js/utils/useTimeEntries.ts Improved type annotations and standardized error handling
resources/js/utils/useCurrentTimeEntry.ts Fixed time entry update to properly handle end times
resources/js/utils/notification.ts Changed login route from route('login') to '/login'
resources/js/packages/ui/tailwind.theme.js New shared Tailwind theme configuration file
resources/js/packages/ui/styles.css New shared CSS styles file
resources/js/packages/ui/src/utils/time.ts Simplified parseTimeInput return type and added HH:MM:SS support
resources/js/packages/ui/src/utils/cn.ts New utility for className merging
resources/js/packages/ui/src/tooltip/* New tooltip component implementation
resources/js/packages/ui/src/index.ts Exported new components and utilities
resources/js/packages/ui/src/Buttons/* Added new Button component with variants
resources/js/packages/ui/src/TimeTracker/* Enhanced with manual entry dropdown and improved range selector
resources/js/packages/ui/src/TimeEntry/* Updated with container queries and improved spacing
resources/js/packages/ui/src/FullCalendar/* Added activity status plugin and running time entry support
resources/js/packages/ui/package.json Version bump and new dependencies added
resources/js/packages/api/src/openapi.json.client.ts Added employees_can_manage_tasks field to organization schema
resources/js/packages/api/package.json Version bump to 0.0.5
resources/js/Pages/Time.vue Removed manual time entry modal (moved to TimeTracker component)
Files not reviewed (1)
  • resources/js/packages/api/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

StanBarrows and others added 5 commits December 1, 2025 12:52
Bumps the minor-updates group with 1 update in the / directory: [dedoc/scramble](https://github.com/dedoc/scramble).


Updates `dedoc/scramble` from 0.12.36 to 0.13.6
- [Release notes](https://github.com/dedoc/scramble/releases)
- [Commits](dedoc/scramble@v0.12.36...v0.13.6)

---
updated-dependencies:
- dependency-name: dedoc/scramble
  dependency-version: 0.13.6
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: minor-updates
...

Signed-off-by: dependabot[bot] <support@github.com>
…-updates-7b51c39769

Bump dedoc/scramble from 0.12.36 to 0.13.6 in the minor-updates group across 1 directory
@StanBarrows StanBarrows merged commit ae5ce9e into production Dec 1, 2025
10 checks passed
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.

5 participants