Povilas Staskus
Povilas Staskus
> > #### 4th scenario - Log in to an account without any site -> Add Widget -> Log out: > > ``` > > 1. Make sure you are...
1. Merged with the other Stats widget changes and retested the solution 2. Set and removed `WPStatsHomeWidgetsUserDefaultsSiteIdKey` in `StatsWidgetStore` instead of `AccountService`. Since this key is only designed to be...
1. Expanded noSite state to show text based on the widget type 2. Expanded noData state to show text based on the widget type (for the sake of consistency) 3....
@guarani, thanks for testing the case when the site is deleted. I admit I didn't think about it. It a little bit reminds of https://github.com/wordpress-mobile/WordPress-iOS/issues/18914 . I'll explore at least...
Thanks again for the comments. It's really an interesting problem how to better structure `StatsWidget` code to react to updates of the app state. The question is which state changes...
> > 5th scenario - Log in into account that has a site -> Remove site > > When I log into an account that has a site, and then...
@guarani Do you know the setup you had or more details about the scenario to reproduce it? > When I log into an account that has a site, and then...
@guarani Should we allow adding widgets when user is **not logged in**, if we **wouldn't allow users without a site** add any of the current widgets? I think we could...
Note: Added a new issue related to account having no site and widgets https://github.com/wordpress-mobile/WordPress-iOS/issues/18914
Note: Consider the opposite case. 1. In the app log in to account that has a site 2. Add a widget 3. Remove a site on the site 4. Return...