Skip to content

Conversation

@joshwanf
Copy link
Collaborator

@joshwanf joshwanf commented Dec 16, 2025

While I'm slowly familiarizing myself with the codebase, I started at the frontend to learn how the data flows. I tried to structure the commits so they are easily reversible. Most changes only affect the main project list page and create/edit project menu (did not dive deeper yet).

Breaking changes

  • Added typescript and prettier as dev tools with strict type checking to tsconfig.json
    • Functions will need to use and return defined types.
    • Will need the prettier extension in VSC to format on save.
  • Commit 388c5f9 set up Vite server proxy
    • Vite has server proxy rules for the dev environment in its config. This removes removes the need to set env variable VITE_API_URL in prod. Dev only needs to be pointed to the API server (e.g., http://localhost:8081).
    • This lets fetch functions in /builder-frontend/src/api simply call /api/resource/....
    • /builder-frontend/.env will need to change VITE_API_URL to just the server path (remove trailing /api).
  • Project/screener components moved into @/components/Project/ProjectDetails with accompanying import changes

Organization changes

  • Commit 13d49da nested routing through <ProtectedRoute>
    Used <ProtectedRoute> as a layout for the main <Route> path "/" to handle checking the auth session and display page layout (if user session is null, show <AuthForm>, otherwise show <Header> and the route outlet)
  • Commit edec3fb
    • Used <Title> for different pages
    • Moved logos into /public for static assets
    • Used for navbar links (SolidJS automatically injects an active/inactive class and simplified navbar generation (only need label text and href, browser <a> handles the rest)
  • Commit 68459f1 split into components , (fetches data and displays cards), and (card displays)

@joshwanf joshwanf marked this pull request as ready for review December 16, 2025 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants