Skip to content
This repository has been archived by the owner on Jun 23, 2023. It is now read-only.

Added user settings handling #250

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

louneskmt
Copy link
Contributor

@louneskmt louneskmt commented Nov 1, 2020

This PR allows custom user settings.

Add store functions to get current settings and update one, using this PR new routes.

Related PR:

EddWills95
EddWills95 previously approved these changes Nov 2, 2020
Copy link
Member

@lukechilds lukechilds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great but will depend on some changes from getumbrel/umbrel-manager#58

@AaronDewes
Copy link
Contributor

@louneskmt This needs rebasing.
@lukechilds This PR makes a few other PRs possible, like #249 and others that aren't created yet. Can you review this one soon?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants