Skip to content

Entities deletion and media cleanup#663

Closed
rajat1saxena wants to merge 5 commits intomainfrom
media-and-db-cleanup
Closed

Entities deletion and media cleanup#663
rajat1saxena wants to merge 5 commits intomainfrom
media-and-db-cleanup

Conversation

@rajat1saxena
Copy link
Copy Markdown
Member

@rajat1saxena rajat1saxena commented Nov 13, 2025

In this change, we have introduced

  1. User deletion (to comply with GDPR)
  2. Communities deletion
  3. Media cleanup of the files added via the text editor

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 13, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
courselit-docs Ready Ready Preview Comment Nov 13, 2025 2:12am

@rajat1saxena rajat1saxena changed the title Entities deletion and media dleanup Entities deletion and media cleanup Nov 13, 2025
github-advanced-security[bot]
github-advanced-security AI found potential problems Nov 13, 2025
View reviewed changes
Comment thread apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx Fixed
Comment thread apps/web/graphql/communities/__tests__/logic.test.ts Fixed
Comment thread apps/web/graphql/users/__tests__/delete-user.test.ts Fixed
Comment thread apps/web/graphql/users/__tests__/delete-user.test.ts Fixed
@rajat1saxena rajat1saxena self-assigned this Nov 13, 2025
github-advanced-security[bot]
github-advanced-security AI found potential problems Nov 13, 2025
View reviewed changes
Comment thread apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx
const trimmedFormName = basicDetails.name.trim();
const formActive = basicDetails.active;
const formTags = basicDetails.tags || [];
const userTags = userData.tags || [];

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused variable userTags.

Copilot Autofix

AI 5 months ago

The fix involves removing the unused variable declaration for userTags from line 401: const userTags = userData.tags || [];. Since this variable is not referenced elsewhere and its removal does not affect the functionality of the code (the correct variables are being used in the logic that follows), deleting this line (and nothing else) resolves the warning. No additional imports, methods, or definitions are required. The change should take place at line 401, within apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx.

Suggested changeset 1
apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx b/apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx
--- a/apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx
+++ b/apps/web/app/(with-contexts)/dashboard/(sidebar)/users/[id]/page.tsx
@@ -398,7 +398,6 @@
     const trimmedFormName = basicDetails.name.trim();
     const formActive = basicDetails.active;
     const formTags = basicDetails.tags || [];
-    const userTags = userData.tags || [];
     // const hasTagsChanged = haveStringArraysChanged(formTags, userTags);
     const hasBasicDetailsChanged =
         trimmedFormName !== initialBasicDetailsRef.current.name ||
EOF
@@ -398,7 +398,6 @@
const trimmedFormName = basicDetails.name.trim();
const formActive = basicDetails.active;
const formTags = basicDetails.tags || [];
const userTags = userData.tags || [];
// const hasTagsChanged = haveStringArraysChanged(formTags, userTags);
const hasBasicDetailsChanged =
trimmedFormName !== initialBasicDetailsRef.current.name ||
Copilot is powered by AI and may make mistakes. Always verify output.
@rajat1saxena rajat1saxena deleted the media-and-db-cleanup branch November 13, 2025 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@rajat1saxena rajat1saxena

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rajat1saxena @github-advanced-security