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 •
edited
Loading

  • 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 added 17 commits January 8, 2026 09:46
@Daan0709
Revamp backend for file system reading and writing
98ac2de
@Daan0709
Editor now fetches files from actual filesystem
dc59186
@Daan0709
Saving now works again for the editor, now directly writes to filesystem
48e6b80
@Daan0709
Can now save and read directly from local file system
389ebd3
@Daan0709
Updated controllertests
539100a
@Daan0709
Test updates
9171b39
@Daan0709
Pipeline fix?
99c171d
@Daan0709
Added testing for filetreeservice
f18b6c0
@Daan0709
Refactored traverse functions
b2fd17c
@Daan0709
Now caches provider data on first load, and only updates the cache if…
2187a37 
… filetree from backend has changed
@Daan0709
Filetree now requests folderdata when expanded, instead of preloading…
f041e82 
… the entire tree, which was expensive
@Daan0709
Merge branch 'master' into feat/optimize-filetree-request-path
5ba9500
@Daan0709
Merge changes
c1bc0e1
@Daan0709
Pipeline and security fixes
1e7b306
@Daan0709
Added tests
0be8c30
@Daan0709
Added test
1828c9c
@Daan0709
Removed auto load of dummy projects, added duplication protection for…
a750f12 
… projects
@Daan0709 Daan0709 linked an issue Jan 14, 2026 that may be closed by this pull request
Cleanup outdated code #179
Closed
@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
Matthbo
Matthbo reviewed Feb 5, 2026
View reviewed changes
@Daan0709 Daan0709 requested a review from Matthbo February 6, 2026 10:25
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 6, 2026

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
76.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Matthbo Matthbo Matthbo approved these changes

@stijnpotters1 stijnpotters1 Awaiting requested review from stijnpotters1

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Cleanup outdated code

3 participants

@Daan0709 @Matthbo @stijnpotters1