[Feat]: Add Delete Task Timesheet function

Innocent-Akim commented 1 week ago

Description

https://github.com/ever-co/ever-teams/issues/3045 Please include a summary of the changes and the related issue.

Type of Change

[x] Bug fix
[x] New feature
[ ] Breaking change
[ ] Documentation update

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings

Previous screenshots

Please add here videos or images of previous status

Current screenshots

Please add here videos or images of previous status

Summary by CodeRabbit

Release Notes

New Features
- Introduced bulk deletion functionality for task timesheets via a new API endpoint.
- Added a new hook for managing timesheet log deletions, including loading states.
- Implemented a new API function for deleting specific timesheet logs based on IDs.
- Added state management for deleted timesheets within the application.
- Introduced a confirmation dialog component for user actions.
- Enhanced task management with improved filtering options and user interface updates.
Bug Fixes
- Enhanced error handling for deletion requests to ensure proper user feedback.

coderabbitai[bot] commented 1 week ago

Walkthrough

The changes introduce a new asynchronous DELETE endpoint for bulk deletion of task timesheets, alongside a new API function for deleting timesheet logs. The useTimesheet hook is updated to manage the deletion process, incorporating a loading state and a callback function. Additionally, a new type and function for handling deletion requests are added, and a state atom for tracking deleted timesheets is introduced. These modifications enhance the functionality related to managing timesheet logs and their deletion.

Changes

File Path	Change Summary
`apps/web/app/api/timer/timesheet/bulk-delete/route.ts`	Added asynchronous DELETE endpoint for bulk deletion of task timesheets; utilizes `authenticatedGuard` and `deleteTaskTimesheetRequest`.
`apps/web/app/hooks/features/useTimesheet.ts`	Introduced `deleteTaskTimesheetLogsApi` function; updated `useTimesheet` hook to manage deletion state and callback.
`apps/web/app/services/client/api/timer/timer-log.ts`	Added `deleteTaskTimesheetLogsApi` function for deleting timesheet logs; modified import statement to include `deleteApi`.
`apps/web/app/services/server/requests/timesheet.ts`	Introduced `IDeleteTimesheetProps` type and `deleteTaskTimesheetRequest` function for handling deletion requests.
`apps/web/app/stores/time-logs.ts`	Added new atom `timesheetDeleteState` for managing deleted timesheets state.
`apps/web/app/[locale]/timesheet/[memberId]/components/TimesheetAction.tsx`	Added `StatusAction` type for button actions; updated button configurations to reflect new status actions.
`apps/web/app/hooks/features/useTimelogFilterOptions.ts`	Updated to manage `timesheetDeleteState` and added selection handling for timesheet items.
`apps/web/components/ui/alert-dialog.tsx`	Introduced a new alert dialog component using Radix UI for confirmation actions.
`apps/web/lib/components/alert-dialog-confirmation.tsx`	Added `AlertDialogConfirmation` component for handling confirmation dialogs with customizable actions.
`apps/web/lib/components/index.ts`	Exported `AlertDialogConfirmation` component for broader access.
`apps/web/lib/features/integrations/calendar/table-time-sheet.tsx`	Updated to manage task deletions and integrate confirmation dialogs within the `DataTableTimeSheet` component.
`apps/web/lib/features/user-profile-plans.tsx`	Enhanced filtering options and layout adjustments for managing user profile plans.
`apps/web/package.json`	Added `@radix-ui/react-alert-dialog` dependency; reorganized existing dependencies.

Possibly related PRs

#3159: The changes in this PR refine the logic for starting the timer based on user interactions with the "Plan Today" popup, which is directly related to the functionality of the new DELETE endpoint in the main PR that manages task timesheet deletions.
#3161: This PR modifies the useStartStopTimerHandler function to enforce plan requirements, which aligns with the authentication and validation logic introduced in the main PR for deleting timesheets.
#3254: The introduction of the getTaskTimesheetLogsApi function in this PR enhances the API integration for timesheet logs, which complements the new DELETE endpoint functionality in the main PR.
#3266: This PR introduces a grouped display for timesheet data by date, which relates to the overall management of timesheet entries, including deletions as handled in the main PR.
#3305: The changes in this PR enhance the TimeSheetFilterPopover component, which is relevant to the filtering and management of timesheet entries, including those that may be deleted as per the main PR.
#3322: This PR modifies the UserProfilePlans component to manage daily plans, which is relevant to the overall timesheet management system that includes deletion functionality as introduced in the main PR.
#3333: This PR introduces Timelog FilterOptions, which enhances the filtering capabilities for timesheet entries, directly relating to the management and deletion of those entries as handled in the main PR.

Suggested labels

feature, WEB, Ever Teams

Suggested reviewers

evereq
Cedric921

Poem

🐇 In the meadow, where timesheets lay,
A new path opens, bright as day.
With logs to delete, and states to track,
Our bunny team hops, never looking back!
So gather 'round, let’s cheer and play,
For the timesheet magic is here to stay! 🌼

[!WARNING] There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure.

🔧 eslint
> If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

apps/web/app/hooks/features/useTimelogFilterOptions.ts
Oops! Something went wrong! :( ESLint: 8.46.0 ESLint couldn't find the config "next/core-web-vitals" to extend from. Please check that the name of the config is correct. The config "next/core-web-vitals" was referenced from the config file in "/apps/web/.eslintrc.json". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

- [X](https://twitter.com/intent/tweet?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A&url=https%3A//coderabbit.ai) - [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai) - [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai) - [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)

🪧 Tips

### Chat There are 3 ways to chat with [CodeRabbit](https://coderabbit.ai): - Review comments: Directly reply to a review comment made by CodeRabbit. Example: - `I pushed a fix in commit , please review it.` - `Generate unit testing code for this file.` - `Open a follow-up GitHub issue for this discussion.` - Files and specific lines of code (under the "Files changed" tab): Tag `@coderabbitai` in a new review comment at the desired location with your query. Examples: - `@coderabbitai generate unit testing code for this file.` - `@coderabbitai modularize this function.` - PR comments: Tag `@coderabbitai` in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples: - `@coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.` - `@coderabbitai read src/utils.ts and generate unit testing code.` - `@coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.` - `@coderabbitai help me debug CodeRabbit configuration file.` Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. ### CodeRabbit Commands (Invoked using PR comments) - `@coderabbitai pause` to pause the reviews on a PR. - `@coderabbitai resume` to resume the paused reviews. - `@coderabbitai review` to trigger an incremental review. This is useful when automatic reviews are disabled for the repository. - `@coderabbitai full review` to do a full review from scratch and review all the files again. - `@coderabbitai summary` to regenerate the summary of the PR. - `@coderabbitai resolve` resolve all the CodeRabbit review comments. - `@coderabbitai configuration` to show the current CodeRabbit configuration for the repository. - `@coderabbitai help` to get help. ### Other keywords and placeholders - Add `@coderabbitai ignore` anywhere in the PR description to prevent this PR from being reviewed. - Add `@coderabbitai summary` to generate the high-level summary at a specific location in the PR description. - Add `@coderabbitai` anywhere in the PR title to generate the title automatically. ### CodeRabbit Configuration File (`.coderabbit.yaml`) - You can programmatically configure CodeRabbit by adding a `.coderabbit.yaml` file to the root of your repository. - Please see the [configuration documentation](https://docs.coderabbit.ai/guides/configure-coderabbit) for more information. - If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: `# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json` ### Documentation and Community - Visit our [Documentation](https://docs.coderabbit.ai) for detailed information on how to use CodeRabbit. - Join our [Discord Community](http://discord.gg/coderabbit) to get help, request features, and share feedback. - Follow us on [X/Twitter](https://twitter.com/coderabbitai) for updates and announcements.

socket-security[bot] commented 1 week ago

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report↗︎

socket-security[bot] commented 1 week ago

Report too large to display inline

View full report↗︎

ever-co / ever-teams