feat(app): implement account and public profile in settings by Carter907 · Pull Request #218 · bluelearn-org/bluelearn

Carter907 · 2026-07-30T16:01:17Z

Summary

This PR is based on PR 212.

Updated UI: Make email and password changes expand inline for a more clean, condensed UI layout
Auth Sync: Used existing auth session with effect hook to populate email field with current email
Update Email: Added functionality for updating email using updateEmail from @/lib/auth
Update/Change Password: Added supabase API integration for changing your password from @/lib/auth
Permanent Account Deletion Dialog: On account deletion, users are told to type "delete account" in a popup dialog to confirm this destructive action.
Profile Page Updates: Added data fetching and notification via a toast when you save changes.

closes #50

Type of change

Verification

pnpm -r typecheck passes
pnpm -r build passes
Manually verified in a browser (for UI / behavior changes)
Followed the app layout conventions
No new dependencies, or new dependencies are justified and AGPL-compatible
Change does not violate any non-negotiable principle
Documentation only, no changes to the codebase, so no tests required

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

…arn into fix/settings-page-1

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Signed-off-by: Tonycs <lam.tony540@gmail.com>

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Signed-off-by: Tonycs <lam.tony540@gmail.com>

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

…arn into fix/settings-page-1

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Signed-off-by: Tonycs <lam.tony540@gmail.com>

…arn into fix/settings-page-1

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Signed-off-by: Carter <speers.carter@gmail.com>

andyrodrigues30

Looks like it's still in the process of the functionality being added, but see comments made on the UI stuff.
Have a look at the error in the ThemeProvider please and see if you can fix it/ensure changing stuff does break anything else.

andyrodrigues30 · 2026-07-30T18:38:12Z

+              {!isEmailEditing && (
+                <Button
+                  variant="outline"
+                  onClick={() => setIsEmailEditing(true)}


Add the following to the button:
className="btn-sec"

andyrodrigues30 · 2026-07-30T18:39:36Z

+                  )}
+                  <Button
+                    variant="ghost"
+                    onClick={() => {


Add the following to the button:
className="btn-sec"

andyrodrigues30 · 2026-07-30T18:39:44Z

+                  >
+                    Cancel
+                  </Button>
+                  <Button onClick={handleSave} disabled={saving}>


Add the following to the button:
className="btn-pri"

andyrodrigues30 · 2026-07-30T18:40:08Z

+              </div>
+              {!isPasswordEditing && (
+                <Button
+                  variant="outline"


Add the following to the button:
className="btn-sec"

andyrodrigues30 · 2026-07-30T18:40:19Z

+                    <p className="mono-micro text-destructive">{updateError}</p>
+                  )}
+                  <Button
+                    variant="ghost"


Add the following to the button:
className="btn-sec"

andyrodrigues30 · 2026-07-30T18:40:28Z

+                  >
+                    Cancel
+                  </Button>
+                  <Button onClick={handleUpdate} disabled={updating}>


Add the following to the button:
className="btn-pri"

andyrodrigues30 · 2026-07-30T18:43:48Z

+const ThemeContext = createContext<ThemeContextType | null>(null);
+
+export function ThemeProvider({ children }: { children: ReactNode }) {
+  const [theme, setThemeState] = useState<Theme>("light");


change theme to themeState
check if this causes any other issues when changing it.

andyrodrigues30 · 2026-07-30T18:46:09Z

+            </div>
+
+            {isEmailEditing && (
+              <div className="space-y-4 rounded-md border p-4">


remove the border and rounded corners

andyrodrigues30 · 2026-07-30T18:46:45Z

+            </div>
+
+            {isPasswordEditing && (
+              <div className="space-y-4 rounded-md border p-4">


remove border and rounded corners

andyrodrigues30 · 2026-07-30T18:47:43Z

+        </p>
+        <Button
+          variant="destructive"
+          className="font-mono tracking-[0.08em] uppercase"


add functionality to delete account

Signed-off-by: Carter <speers.carter@gmail.com>

…at/settings-page-2

Adds client-side validation and old password verification using Supabase before updating to the new password. Signed-off-by: Carter <speers.carter@gmail.com>

Signed-off-by: Carter <speers.carter@gmail.com>

opted for TanStack Router loader pattern settings.account.tsx with getMyIdentity, replacing the client-side useAuth hook for fetching initial data. Signed-off-by: Carter <speers.carter@gmail.com>

squeaky-duck-duck and others added 30 commits July 20, 2026 11:45

fix(app): apply requested changes to /settings

54ffc52

Merge branch 'main' into fix/settings-page-1

9e7737c

fix: api client import

72e1187

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

feat(app): create settings route page

daa0f18

feat(app): settings/account route

9ad6d4e

feat(app): create settings index

20c682d

feat(app): create settings/policies route

807df7e

feat(app): settings/appearance route

6a4605d

Merge branch 'main' into fix/settings-page-1

dc29d96

fix: settings page styling and layout issues

9ffbbea

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

fix: theme preview colors

ab31336

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

feat: remove email onChange functionality

96ff681

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

fix: update comment on settings account page

5e29312

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

fix: settings tab styling

ed3e0de

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

fix: padding on policy links in settings

956f082

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Merge branch 'main' into fix/settings-page-1

9829ce2

fix: update imports

76ea38a

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Merge branch 'fix/settings-page-1' of github.com:bluelearn-org/bluele…

e0a2223

…arn into fix/settings-page-1

chore: remove unused import

b59104c

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Merge branch 'main' into fix/settings-page-1

3d92366

feat: remove required from email field and label

d41125b

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Merge branch 'main' into fix/settings-page-1

2201653

feat(api): add email to me routes

a3aa857

Signed-off-by: Tonycs <lam.tony540@gmail.com>

feat(api): add delete user route

02fa306

Signed-off-by: Tonycs <lam.tony540@gmail.com>

feat(app): add deleteMyAccount client function

cf48a1d

Signed-off-by: Tonycs <lam.tony540@gmail.com>

feat: add avatar to settings account page

4b28c71

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

fix(api): update delete account comment

3bd943c

Signed-off-by: Tonycs <lam.tony540@gmail.com>

feat: remove settings policy page

2bd7811

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

feat: fix sidebar scroll

54f256d

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

Merge branch 'fix/settings-page-1' of github.com:bluelearn-org/bluele…

7a17030

…arn into fix/settings-page-1

andyrodrigues30 and others added 7 commits July 29, 2026 19:57

chore: remove unused import

0dfd403

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

feat: add updateEmail function

c3cfbe4

Signed-off-by: Tonycs <lam.tony540@gmail.com>

Merge branch 'fix/settings-page-1' of github.com:bluelearn-org/bluele…

aa219d2

…arn into fix/settings-page-1

feat: separate account and profile pages

f8d58fd

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

feat: add email and password states

f60a77d

Signed-off-by: andyrodrigues30 <andrea.melissa.rodrigues@gmail.com>

feat(app): make email and password forms expand inline

036dece

Signed-off-by: Carter <speers.carter@gmail.com>

feat(app): auto-populate email from auth session

6787d71

Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 requested review from andyrodrigues30 and tonylam0 as code owners July 30, 2026 16:01

feat(app): hook up change email functionality to Supabase API

6916283

Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 force-pushed the feat/settings-page-2 branch from 6603976 to 6916283 Compare July 30, 2026 18:20

Carter907 marked this pull request as draft July 30, 2026 18:27

andyrodrigues30 requested changes Jul 30, 2026

View reviewed changes

Carter907 added 2 commits July 30, 2026 15:23

fix(app): applied review style changes

fe17420

Signed-off-by: Carter <speers.carter@gmail.com>

Merge branch 'main' of https://github.com/Carter907/bluelearn into fe…

d09e3fd

…at/settings-page-2

Carter907 force-pushed the feat/settings-page-2 branch from 5eb1ea7 to d09e3fd Compare July 30, 2026 19:41

Carter907 requested a review from andyrodrigues30 July 30, 2026 20:05

Carter907 added 3 commits July 30, 2026 20:10

feat(app): implement password change integration

b5d549c

Adds client-side validation and old password verification using Supabase before updating to the new password. Signed-off-by: Carter <speers.carter@gmail.com>

feat(app): Implemented account deletion dialog

7aafd2e

Signed-off-by: Carter <speers.carter@gmail.com>

feat(app): Implemented settings profile data fetching

2ba22ba

Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 marked this pull request as ready for review July 31, 2026 01:14

refactor(app): fetch initial account data via loader

7661e04

opted for TanStack Router loader pattern settings.account.tsx with getMyIdentity, replacing the client-side useAuth hook for fetching initial data. Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 force-pushed the feat/settings-page-2 branch from fd00b76 to 7661e04 Compare July 31, 2026 01:30

Merge branch 'bluelearn-org:main' into feat/settings-page-2

59b647d

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(app): implement account and public profile in settings - #218

feat(app): implement account and public profile in settings#218
Carter907 wants to merge 45 commits into
bluelearn-org:mainfrom
Carter907:feat/settings-page-2

Carter907 commented Jul 30, 2026 •

edited

Loading

Uh oh!

andyrodrigues30 left a comment

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

andyrodrigues30 Jul 30, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

Carter907 commented Jul 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of change

Verification

Uh oh!

andyrodrigues30 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!

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Carter907 commented Jul 30, 2026 •

edited

Loading