Skip to content

refactor: clean all the backend#107

Open
Exeloo wants to merge 67 commits into
mainfrom
refactor/clean-all-backend
Open

refactor: clean all the backend#107
Exeloo wants to merge 67 commits into
mainfrom
refactor/clean-all-backend

Conversation

@Exeloo
Copy link
Copy Markdown
Member

@Exeloo Exeloo commented Apr 28, 2026

What does this PR do?

How do you test this PR?

@Exeloo Exeloo requested a review from bill-h4rper as a code owner April 28, 2026 05:49
@Exeloo Exeloo marked this pull request as draft April 28, 2026 05:49
Comment thread src/lib/server/session/session-functions.ts Outdated
Comment thread src/lib/server/session/session.const.ts Outdated
Comment thread src/lib/server/session/project/project-functions.ts Outdated
@Exeloo Exeloo requested a review from Tchips46 May 6, 2026 13:01
@Exeloo Exeloo marked this pull request as ready for review May 6, 2026 13:01
bill-h4rper
bill-h4rper previously approved these changes May 6, 2026
Comment thread src/lib/server/api/base.repository.ts Outdated
Comment thread src/lib/server/file-system/project-file.ts
Comment thread src/lib/server/loader/env.ts Outdated
Comment thread src/lib/utils/types/save.type.ts
Comment thread src/lib/utils/string.ts Outdated
bill-h4rper
bill-h4rper previously approved these changes May 11, 2026
Comment thread src/lib/server/actions/project/new.action.ts
Comment thread src/lib/server/actions/project/cli/add-system.action.ts Outdated
Comment thread src/lib/server/actions/project/cli/add-system.action.ts Outdated
@Exeloo
Copy link
Copy Markdown
Member Author

Exeloo commented May 22, 2026

Todo

  • Handle body validation with class validator (another PR)
  • Add a route to validate a project before loading (another PR)
  • Add /load page that allow to load project from query params (another PR)
  • Handle hash for cache files and add invalidate cache on load with differents files with server (another PR)
  • Handle file writting on code editor (another PR)

Tchips46
Tchips46 previously approved these changes May 26, 2026
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.

3 participants