> > 36 if (status.service_enabled)
> > 37 {
> > 38 - service_enabled = true;
> > 39 - break;
> > 40 + return true;
> > 41 }
> >
> > Just "return status.service_enabled" ?
>
> That if statement is within a loop over all accounts. So default is false, the
> if one is enabled return true
Ah, ok, that wasn't obvious from the diff. Looks good now, thanks!
> > 36 if (status. service_ enabled) service_ enabled" ?
> > 37 {
> > 38 - service_enabled = true;
> > 39 - break;
> > 40 + return true;
> > 41 }
> >
> > Just "return status.
>
> That if statement is within a loop over all accounts. So default is false, the
> if one is enabled return true
Ah, ok, that wasn't obvious from the diff. Looks good now, thanks!