Skip to content

Jimqueline/backend refactor first#72

Merged
zijin1994 merged 16 commits into
mainfrom
jimqueline/backend-refactor-first
Apr 20, 2025
Merged

Jimqueline/backend refactor first#72
zijin1994 merged 16 commits into
mainfrom
jimqueline/backend-refactor-first

Conversation

@zijin1994
Copy link
Copy Markdown
Contributor

First pass of backend refactor
followed this pattern:

  1. move route logic into utils folder next to the route.
  2. for non api call functions, they sit inside the utils.ts file in the utils folder.
  3. if there are more than one type definitions, they go into a types.ts file next to the route, otherwise, they sit next to the function.

If we want to further refactor the code, start moving functions inside the utils functions near the route.

Copy link
Copy Markdown
Contributor

@jq-ln jq-ln left a comment

Choose a reason for hiding this comment

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

Looks good, works for me!

@zijin1994 zijin1994 merged commit 108a245 into main Apr 20, 2025
3 checks passed
@zijin1994 zijin1994 deleted the jimqueline/backend-refactor-first branch April 20, 2025 19:41
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