AmpliPi icon indicating copy to clipboard operation
AmpliPi copied to clipboard

More Admin Settings

Open SteveMicroNova opened this issue 1 year ago • 2 comments

What does this change intend to accomplish?

Builds on #828's Admin Tab on the updater and adds more settings, including the option to set timezone (closes #787) and change minimum log level

Blocked by the merging of #828, creating a draft PR just to toss this into the public eye while I await review there

Checklist

  • [x] Have you tested your changes and ensured they work?
  • [x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • [ ] If applicable, have you updated the documentation/manual?
  • [x] If applicable, have you updated the CHANGELOG?
  • [x] Does your submission pass linting & tests? You can test on localhost using ./scripts/test
  • [ ] Have you written new tests for your core features/changes, as applicable?
  • [x] If this is a UI change, have you tested it across multiple browser platforms?
  • [x] If this is a UI change, have you tested across multiple viewport sizes (ie. desktop versus mobile)?

SteveMicroNova avatar Aug 08 '24 17:08 SteveMicroNova

image

SteveMicroNova avatar Aug 08 '24 17:08 SteveMicroNova

I am marking this PR for death, it has reached a state where it would be better to manually move these changes over to another branch by hand due to the advancements on both main and the underlying branch that is #828

Once #828 is merged, I'll be making a new branch off of main and reimplementing this with simple copy+paste and edit and then closing this PR

SteveMicroNova avatar Oct 04 '24 16:10 SteveMicroNova

Closing as dead in the water, cleaning up the "open PRs" view. I can easily return to these notes in the "Closed PRs" view, so nothing is lost.

SteveMicroNova avatar Dec 09 '24 16:12 SteveMicroNova