beeminder / BeeSwift

Official Beeminder for iOS app
Other
29 stars 5 forks source link

Refreshing gaols should only trigger a graph refresh for autodata goals #446

Closed theospears closed 4 months ago

theospears commented 4 months ago

Previously we would trigger a graph refresh on reload for all goals. This wans't a big deal in the past because the UI didn't differentiate that state, but now we show a lemniscate for queued goals it means all pulls to refresh would trigger a lemniscate, making the refresh operation take several seconds to quiesce.

Now instead only trigger this refresh on goals with autodata, so manual goals refresh quicker.

Testing: Refreshed a goal with autodata and saw the lemniscate Refreshed a goal with manual data and saw no lemniscate

theospears commented 4 months ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @theospears and the rest of your teammates on Graphite Graphite