Skip to content

Remove legacy code#178

Merged
Daan0709 merged 33 commits intomasterfrom
remove-legacy-code
Feb 6, 2026
Merged

Remove legacy code#178
Daan0709 merged 33 commits intomasterfrom
remove-legacy-code

Conversation

@Daan0709
Copy link
Contributor

@Daan0709 Daan0709 commented Jan 14, 2026

  • Removed the now outdated Open project button which copied the entire project to memory
  • Added better error handling for creating a duplicate project
  • Removed test configurations from resources folder, which were not accessible with the current code anyway

@Daan0709 Daan0709 linked an issue Jan 14, 2026 that may be closed by this pull request
@Daan0709 Daan0709 marked this pull request as ready for review January 30, 2026 14:48
@Daan0709 Daan0709 marked this pull request as draft January 30, 2026 14:49
@Daan0709 Daan0709 marked this pull request as ready for review February 5, 2026 09:34
@Daan0709 Daan0709 requested a review from Matthbo February 6, 2026 10:25
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 6, 2026

@Daan0709 Daan0709 merged commit ea5748b into master Feb 6, 2026
5 checks passed
@Daan0709 Daan0709 deleted the remove-legacy-code branch February 6, 2026 10:42
VincentvanMedenbach pushed a commit that referenced this pull request Feb 11, 2026
* Revamp backend for file system reading and writing

* Editor now fetches files from actual filesystem

* Saving now works again for the editor, now directly writes to filesystem

* Can now save and read directly from local file system

* Updated controllertests

* Test updates

* Pipeline fix?

* Added testing for filetreeservice

* Refactored traverse functions

* Now caches provider data on first load, and only updates the cache if filetree from backend has changed

* Filetree now requests folderdata when expanded, instead of preloading the entire tree, which was expensive

* Merge changes

* Pipeline and security fixes

* Added tests

* Added test

* Removed auto load of dummy projects, added duplication protection for projects

* Reused sorting function instead of duplicating

* Fixed merge issues

* Removed unused function

* Removed accidental duplicated code

* Removed unnecessary stub from testing

* Added test for projectAlreadyExistsException

* Linter fix

* Now loads root on construction of editorfilesdataprovider, fixed some merge issues

* Error when creating a new project is now caught and displayed in the toast container

* Added some documentation and updated call to backend to make use of service
VincentvanMedenbach pushed a commit that referenced this pull request Feb 11, 2026
* Revamp backend for file system reading and writing

* Editor now fetches files from actual filesystem

* Saving now works again for the editor, now directly writes to filesystem

* Can now save and read directly from local file system

* Updated controllertests

* Test updates

* Pipeline fix?

* Added testing for filetreeservice

* Refactored traverse functions

* Now caches provider data on first load, and only updates the cache if filetree from backend has changed

* Filetree now requests folderdata when expanded, instead of preloading the entire tree, which was expensive

* Merge changes

* Pipeline and security fixes

* Added tests

* Added test

* Removed auto load of dummy projects, added duplication protection for projects

* Reused sorting function instead of duplicating

* Fixed merge issues

* Removed unused function

* Removed accidental duplicated code

* Removed unnecessary stub from testing

* Added test for projectAlreadyExistsException

* Linter fix

* Now loads root on construction of editorfilesdataprovider, fixed some merge issues

* Error when creating a new project is now caught and displayed in the toast container

* Added some documentation and updated call to backend to make use of service
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.

Cleanup outdated code

3 participants