Skip to content

Codebase Refactor (we are not blowing up the entire thing) #156

@FegelSamuel

Description

@FegelSamuel

Description:
Taz and I have drafted some specific TODO's for making the codebase less awful and AI-generated. A good chunk of these need to be completed before you start working on anything else (like any frontend issue. all frontend is blocked until you unfuck the frontend)

TODO:

  • Refactor the /server/api directory to fit h3 routing better. See Taz or Samuel for more information
  • Remove all mentions of passwords
  • Double check that DateTimes are enforced at both API level and Database level
  • Delete all migrations directories and reinitialize Prisma
  • Everything except the README.md from the root directory should be deleted. Move the nuxt project to the root directory. Basically, you should be able to run npm run dev without having to cd into mkckiefwogublnuxtapp whatever dierctory
  • Convert all js files to ts. We like typechecking.
  • Use $fetch() or useFetch() instead of fetch() (useFetch for rendering and $fetch for user action)
  • Redo the ER diagram. It looks like shit. There are also two ER diagrams. Strange
  • Docker needs to be cleaned up or removed, depending on how fast you want to be deployed in production.
  • No watchEffect() allowed. Explicitly specify what you're watching with watch(). Also, watchEffect() stinks of React :skull face emoji:

Sub-issues

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions