Skip to content

Small fixes: Profile delete ReferenceError in finally, dead 'Remember me', undefined Tailwind classes, light-theme ChangePassword #9

Description

@KrishP147

From code audit, batch of low-severity fixes:

  1. frontend/src/pages/Profile.jsx:~325 — finally { if (!error) ... } references error not in scope → ReferenceError / unhandled rejection on every account-delete run; modal reset never runs. Fix: local let succeeded=false flag.
  2. frontend/src/pages/Login.jsx:~37-45 — "Remember me for 30 days" is passed via options.data, which signInWithPassword ignores. Fix: remove the checkbox, or implement via auth.storage sessionStorage/localStorage switch in supabaseClient.js.
  3. frontend/src/App.jsx:~192-193 — loading screen uses bg-dark-primary, text-matrix-green-400 which don't exist in tailwind.config.js. Fix: bg-black / text-primary-500.
  4. frontend/src/pages/ChangePassword.jsx:~87-160 — still the old light theme (purple gradient, bg-white, text-gray-*). Restyle with .card / .input to match the dark app. Minor: MealList.jsx:~730 text-gray-500.

Acceptance: lint + tests green; Profile delete flow test covers the finally path.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions