beeminder / BeeSwift

Official Beeminder for iOS app
Other
30 stars 6 forks source link

Reduce CurrentUserManager wrappers #520

Closed theospears closed 4 days ago

theospears commented 4 days ago

Previously many attributes on User were read using wrappers in CurrentUserManager. These wrappers would do lots of context allocation. Get rid of the wrappers, and have the end users just read from the User object directly.

Testing: Verified that it was still possible to change goal notification settings

theospears commented 4 days ago

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