Neu folder handler#8
Open
noschXL wants to merge 12 commits into
Open
Conversation
crispy-caesus
requested changes
Oct 20, 2025
crispy-caesus
left a comment
crispy-caesus
left a comment
Member
There was a problem hiding this comment.
brauchen wir das ganze lua zeugs noch oder ist das ne config die nicht geht?
Member
There was a problem hiding this comment.
move a lot of the installuserconfig stuff into seperate functions, this is hard to read
optimally you would have the install function pretty much only have descriptive calls to functions so you can see
- getting user input
- checking config selection
- etc
which would make the code a lot easier to navigate for new people/if you haven't worked on it in a long time
Member
There was a problem hiding this comment.
also split up functions to create, move, perms seperately
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
a python script to make and handle the members
(/media/share/roboag/User/...)
also added a fix to the userconfiginstaller, now the config gets chowned by the user the config is installed for