Update documentations + Added Delete Account Feature tested manually by khanhdo05 · Pull Request #127 · StressLess-Team/StressLess

khanhdo05 · 2025-05-13T22:43:55Z

Update docs

Added info about database and firebase
Added info about contributing

Delete Account Feature

Added a controller for deleting account that delete all instances of user_id across tables, then finally the user
Frontend: handle deleting user account from Firebase auth, handling auth context, added button and dialog for deleting account

How to test

Run local host
Sign up a new account, put in preferences info, add a bunch of events and deadlines, generate events.
Go to /dashboard/profile and click Delete account -> Yes, delete.

Expected behavior

Got rerouted to / and not signed-in any more.
Check Neon and Firebase tables and see that your account and associated data are all deleted.

…ester is over into readme

Added information about providing own database and Firebase after grading is over into readme

Events include deadlines id

khanhdo05 · 2025-05-14T13:39:42Z

backend/src/controllers/DeadlineController.ts

This is just fixing typos

trappaly · 2025-05-14T19:57:36Z

README.md

+How to set up the database will depend on whether the database services used for our development are still available:
+- Before Spring 2025 grades are released, you may ask a team member for credentials.
+  - This way, graders may use the team's credentials for grading purposes.
+- After grades are released, please provide your own PostgreSQL database and Firebase credentials.


Added descriptions on setting up database

trappaly · 2025-05-14T19:58:14Z

README.md

+Ensure you are in the `backend` directory (which you should be if you completed the previous step).
+
+**To add the schema to the development database:**
+```bash


Instructions on how to add scheme to database

trappaly · 2025-05-14T19:58:49Z

backend/src/controllers/AuthController.ts

+   * @param res
+   * @constructor
+   */
+  public static async deleteAccount(req: Request, res: Response): Promise<any> {


Deleting account by deleting all of the users preferences from the database

trappaly · 2025-05-14T19:59:52Z

docs/dev docs/contributing.md

@@ -0,0 +1,62 @@
+# Contributing
+


Detailed instructions on how future users can contribute to our repository

trappaly · 2025-05-14T20:00:22Z

docs/dev docs/database/Database Schema Design.md

@@ -2,7 +2,7 @@



Updated database schema

trappaly · 2025-05-14T20:02:00Z

frontend/src/components/UserProfile.tsx

 export default function ProfilePage() {
-  const { user, loading } = useAuth();
+  const { user, deleteAccount, loading } = useAuth();
  // Using state to hold user preferences


Added delete account to the profile page

trappaly · 2025-05-14T20:02:39Z

frontend/src/components/UserProfile.tsx

+      if (!user?.uid) return;
+
+      // Delete user data from backend
+      const res = await axios.delete(


Deletes the user from both the database and firebase

trappaly · 2025-05-14T20:03:48Z

frontend/src/components/UserProfile.tsx

+
+          {/* Delete Account Button */}
+          <Dialog open={openDelete} onOpenChange={setOpenDelete}>
+            <DialogTrigger asChild>


Adds a delete account button and makes sure it's deleted

trappaly · 2025-05-14T20:04:17Z

frontend/src/components/context/auth/AuthContext.tsx

+    }
+    console.log(
+      'Deleting user: ' + user.displayName + ' with uid: ' + user.uid
+    );


Deletes the user's account

trappaly · 2025-05-14T20:04:54Z

frontend/src/components/context/auth/AuthContext.tsx


+  const deleteAccount = async () => {
+    if (!user) {
+      return () => {


Tells which user they're deleting

trappaly

Overall very throughout and detailed code. You also wrote very detailed descriptions and instructions in the README as well. You added a license, you added both the implementation and the frontend buttons for the user to delete an account, and updated the database schema.

As I said on discord, when I ran development it wouldn't delete my account. It would probably be a good idea for other teammates to check this though to see if it's a problem with my computer or with the code itself before I merge the pull request.

trappaly · 2025-05-14T21:17:45Z

frontend/src/components/ReauthenticateModal.tsx

@@ -0,0 +1,84 @@
+'use client';
+
+import { useState } from 'react';


Updated reauthentication model so if you are signed in for a long time firebase will reauthenticate

trappaly · 2025-05-14T21:18:36Z

frontend/src/components/UserProfile.tsx

+              )}
+              {showReauthModal && (
+                <div className="mt-4 border rounded p-4 bg-muted">
+                  <ReauthenticateModal


Added the reauthenticate model button if you're signed into your account for a long time to reauthenticate

trappaly

Looks good! I tested it and am able to delete my account now. I will merge the pull request.

tyu012 and others added 9 commits May 12, 2025 14:30

Added information about providing own database and Firebase after sem…

dd49cfb

…ester is over into readme

Merge pull request #125 from StressLess-Team/feat/readme-db-update

461c270

Added information about providing own database and Firebase after grading is over into readme

Mark milestones completed

f8b7cfa

Update Database Schema Design.md

169b658

Events include deadlines id

add instruction for contributing part 1

9d1874e

Instructions for contributing as an outsider

5932811

Update clone command

6efc4ce

Added: Delete Account Feature

ac0d76b

Create LICENSE

b0f5297

khanhdo05 commented May 14, 2025

View reviewed changes

backend/src/controllers/DeadlineController.ts

Copy link
Copy Markdown

Collaborator Author

khanhdo05 May 14, 2025

Choose a reason for hiding this comment

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

This is just fixing typos

trappaly reviewed May 14, 2025

View reviewed changes

docs/dev docs/database/Database Schema Design.md

@@ -2,7 +2,7 @@

Copy link
Copy Markdown

Collaborator

trappaly May 14, 2025

Choose a reason for hiding this comment

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

Updated database schema

trappaly reviewed May 14, 2025

View reviewed changes

khanhdo05 added 3 commits May 14, 2025 15:56

fixed reauthenticate issue

bc673cd

Merge remote-tracking branch 'origin/development' into development

33aa5c9

fix lint

e65405a

trappaly reviewed May 14, 2025

View reviewed changes

trappaly merged commit 4ab9723 into main May 14, 2025
2 checks passed

		@@ -0,0 +1,84 @@
		'use client';

		import { useState } from 'react';

Conversation

khanhdo05 commented May 13, 2025

Update docs

Delete Account Feature

How to test

Expected behavior

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

trappaly May 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

trappaly left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

trappaly left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

trappaly May 14, 2025 •

edited

Loading