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:45
@StanBarrows StanBarrows merged commit 561ad40 into main Dec 1, 2025
7 of 10 checks passed
@StanBarrows StanBarrows deleted the feature-update branch December 1, 2025 11:48
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 permission system for employees to manage tasks based on an organization setting (employees_can_manage_tasks), refactors the UI package structure for better code sharing between the main app and desktop app, and includes various component improvements and dependency updates.

Key Changes:

  • Adds employees_can_manage_tasks organization setting with comprehensive permission checks
  • Refactors Tailwind theme configuration into a shared package for reusability
  • Adds new UI components (Tooltip, Button variants, TimeTrackerMoreOptionsDropdown)
  • Updates API client with new organization fields
  • Improves time entry handling with better support for running timers

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
tests/Unit/Service/PermissionStoreTest.php Adds comprehensive tests for employee task permission system
tests/Unit/Endpoint/Api/V1/TaskEndpointTest.php Updates tests to use :all permissions and adds employee permission tests
tests/Unit/Endpoint/Api/V1/ClientEndpointTest.php Adds clients:view:all permission to existing tests
tests/Unit/Console/KernelTest.php New test file for console kernel scheduling
tsconfig.json Adds path mapping for @solidtime/ui package
tailwind.config.js Refactors theme into shared package
resources/js/packages/ui/tailwind.theme.js New shared theme configuration
resources/js/packages/ui/styles.css New shared CSS styles
resources/js/utils/useTimeEntries.ts Refactors store with explicit return types and improved error handling
resources/js/utils/useCurrentTimeEntry.ts Adds support for stopping timer by setting end time
resources/js/utils/notification.ts Changes route('login') to hardcoded '/login'
Various UI components New components and refactored existing ones for better modularity
Package files Version bumps and dependency updates
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.

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