Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Settings Management #59

Closed
wants to merge 3 commits into from

Conversation

TanmayArya-1p
Copy link
Contributor

@TanmayArya-1p TanmayArya-1p commented Dec 16, 2024

Closes:
Issue #66

Type of Change

  • Feature

Different parts of the program can now store static settings using the SettingsManager object created in main.py. the settings are stored in config.json in the form of string:string key value pairs. config.json is structured to have a user-defined section which the user can edit and a application section which can be used for more permanent settings.

Other people can now use this for storing their configuration options inside the program.
Also the user can edit the user-defined settings using the settings prompt:
image
image

Here the user can temporarily edit the string:string key value pairs and choose to save changes or revert back to the config before the changes were made. Alternatively the user can also load settings from a URL allowing for config sharing between users.

@IronJam11
Copy link
Collaborator

Kindly mention the issue you are trying to address as well.

@fillingtothemomo
Copy link
Collaborator

@TanmayArya-1p please mention which issue this pr closes and if it is a new feature you proposed , first raise a issue.

@TanmayArya-1p
Copy link
Contributor Author

@fillingtothemomo I have created the issue( #66 ) and mentioned it in the PR

@fillingtothemomo
Copy link
Collaborator

@TanmayArya-1p thanks for working on this, however, i don't see a grave need for this feature especially when it makes a huge change and would meddle with other issues.

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