feat(platform): Implement delete project

[kriptonian1]

User description

Description

This pull request introduces several changes to the apps/platform project, focusing on adding a delete project feature, improving the layout, and optimizing project fetching. The most important changes include the addition of a delete project confirmation dialog, updates to the project card component to support project deletion, and the use of useCallback for fetching projects.

Delete Project Feature:

• New Component: Added ConfirmDeleteProject component to handle project deletion confirmation (apps/platform/src/components/dashboard/project/confirmDeleteProject/index.tsx).
• Project Card: Updated ProjectCard to include delete project functionality and handle the delete project dialog (apps/platform/src/components/dashboard/project/projectCard/index.tsx). [1] [2] [3] [4]

Layout Improvements:

• Navbar and Sidebar Layout: Improved the layout structure by reordering class names for better readability and consistency (apps/platform/src/app/(main)/layout.tsx). [1] [2]

Project Fetch Optimization:

• useCallback: Refactored project fetching logic to use useCallback for better performance and cleaner code (apps/platform/src/app/(main)/page.tsx). [1] [2]
• Delete Project Dialog: Integrated ConfirmDeleteProject into the main page to handle project deletion (apps/platform/src/app/(main)/page.tsx). [1] [2]
  

Fixes #657

Mentions

@rajdip-b @poswalsameer

Screenshots of relevant screens

Screen.Recording.2025-01-28.at.12.33.02.PM.mov

Developer's checklist

• My PR follows the style guidelines of this project
• I have performed a self-check on my work

If changes are made in the code:

• I have followed the coding guidelines
• My changes in code generate no new warnings
• My changes are breaking another fix/feature of the project
• I have added test cases to show that my feature works
• I have added relevant screenshots in my PR
• There are no UI/UX issues

Documentation Update

• This PR requires an update to the documentation at docs.keyshade.xyz
• I have made the necessary updates to the documentation, or no documentation changes are required.

---

PR Type

Enhancement

---

Description

• Added a delete project feature with confirmation dialog.

• Refactored layout structure for improved readability.

• Introduced useCallback for optimized project fetching.

• Updated project card to support project deletion.

---

Changes walkthrough 📝

Relevant files

Enhancement

layout.tsx Refactor layout structure for readability                                apps/platform/src/app/(main)/layout.tsx Refactored layout structure for better readability. Adjusted class names and indentation. +4/-6      page.tsx Add delete project logic and optimize fetching                      apps/platform/src/app/(main)/page.tsx Added useCallback for fetching projects. Integrated delete project confirmation dialog. Improved project fetching logic with error handling. +37/-30  index.tsx Add delete project confirmation component                                apps/platform/src/components/dashboard/project/confirmDeleteProject/index.tsx Added new component for delete project confirmation. Implemented toast messages for success and error states. Integrated project deletion API call. +133/-0  index.tsx Add delete functionality to project card                                  apps/platform/src/components/dashboard/project/projectCard/index.tsx Updated project card to include delete functionality. Added context menu option for project deletion. Integrated project selection for deletion. +18/-5

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[codiumai-pr-agent-free]

CI Feedback 🧐

(Feedback updated until commit e1a79c0)

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: Validate PR title

Failed stage: Lint PR [❌]

Failed test name: Validate PR title

Failure summary: The action failed because the PR title doesn't follow the semantic versioning convention. The title uses "Feat" but it should be "feat" (lowercase) according to conventional commits format. The correct format should be: "feat(platform): Implement delete project"

Relevant error logs: 1: ##[group]Operating System 2: Ubuntu ... 35: Download action repository 'amannn/action-semantic-pull-request@v5' (SHA:0723387faaf9b38adef4775cd42cfd5155ed6017) 36: Complete job name: Validate PR title 37: ##[group]Run amannn/action-semantic-pull-request@v5 38: with: 39: githubBaseUrl: https://api.github.com 40: env: 41: GITHUB_TOKEN: *** 42: ##[endgroup] 43: ##[error]Unknown release type "Feat" found in pull request title "Feat(platform): Implement delete project".

[codiumai-pr-agent-free]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅ 657 - PR Code Verified Compliant requirements: Add ability to delete a project from the platform Show toast messages for success/error states Requires further human verification: Verify that UI matches Figma design

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Error Handling The delete project function continues execution even after showing error toast. Should return early after error to prevent further execution. if (error) { toast.error('Something went wrong!', { description: ( <p className="text-xs text-red-300"> Something went wrong while deleting the project. Check console for more info. </p> ) }) // eslint-disable-next-line no-console -- we need to log the error console.error(error) } handleClose() Inconsistent Usage Project link uses name in some places and slug in others which could cause routing issues. Should consistently use slug. <Link href={`/project/${name}`}> <ContextMenuItem inset>Open</ContextMenuItem> </Link> <a href={`/project/${slug}`} rel="noopener noreferrer" target="_blank"> <ContextMenuItem inset>Open in new tab</ContextMenuItem>

[codiumai-pr-agent-free]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Possible issue	Prevent concurrent deletion operations Add loading state during project deletion to prevent multiple deletion attempts and provide better user feedback apps/platform/src/components/dashboard/project/confirmDeleteProject/index.tsx [34-44] +const [isDeleting, setIsDeleting] = useState(false) const deleteProject = async () => { - if (selectedProject === null) { - toast.error('No project selected', { - description: ( - <p className="text-xs text-red-300"> - No project selected. Please select a project. - </p> - ) - }) + if (selectedProject === null || isDeleting) { return } + setIsDeleting(true) + try { + // ... rest of the deletion logic + } finally { + setIsDeleting(false) + } Apply this suggestion Suggestion importance[1-10]: 8 Why: Adding loading state during deletion is important for preventing race conditions and duplicate deletion attempts, while also improving user feedback. This is a critical enhancement for data safety and UX.	8
General	Handle clipboard operation failures Add error handling for clipboard operations to prevent silent failures and improve user feedback apps/platform/src/components/dashboard/project/projectCard/index.tsx [42-50] -const copyToClipboard = (): void => { - if (navigator.clipboard) { - navigator.clipboard - .writeText(`${window.location.origin}/project/${slug}`) - .then(() => { - toast.success('Link has been copied to clipboard.') - }) - .catch((error) => { +const copyToClipboard = async (): Promise<void> => { + try { + if (navigator.clipboard) { + await navigator.clipboard.writeText(`${window.location.origin}/project/${slug}`) + toast.success('Link has been copied to clipboard.') + } else { + throw new Error('Clipboard API not supported') + } + } catch (error) { + toast.error('Failed to copy to clipboard') + console.error('Error copying text:', error) + } Apply this suggestion Suggestion importance[1-10]: 6 Why: The improved error handling for clipboard operations provides better user feedback and prevents silent failures, enhancing the reliability of the copy functionality.	6

[rajdip-b]

LGTM

[rajdip-b]

🎉 This PR is included in version 2.10.0-stage.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[poswalsameer]

LFG!

[rajdip-b]

🎉 This PR is included in version 2.10.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀