Improve initial setup and UI reactivity by bjorkert · Pull Request #426 · loopandlearn/LoopFollow

Choose a reason for hiding this comment

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

Code review

Not really able to do a code review - the changes seem appropriate, but not an expert.

Test

I switched between various NS URL for the test. I have:

  • Loop with no treatments
  • Loop from "real" users and test phones
  • Trio from "real" users and test phones

I confirm that the remote configuration is restored to none when switching between URL, so the Alarm icon shows up in the tool bar.

I did not notice a difference when switching between URL for dev or this branch.