You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#1882 adds context to the Owner(ctx) method. In an ideal world profile store implementations should be required to store the canonical "Owner" profile in memory, removing the need for any fetching, and by extension the need for a context argument.
We should audit the codebase for owner mutations, return a copy of the profile.Profile in the ProfileStore.Owner() getter method, and add a spec test that assures ProfileStore implementations behave properly with regard to the owner.
Stretch goal: all profile mutations pass through the profile store.
The text was updated successfully, but these errors were encountered:
#1882 adds context to the
Owner(ctx)
method. In an ideal world profile store implementations should be required to store the canonical "Owner" profile in memory, removing the need for any fetching, and by extension the need for a context argument.We should audit the codebase for owner mutations, return a copy of the profile.Profile in the
ProfileStore.Owner()
getter method, and add a spec test that assures ProfileStore implementations behave properly with regard to the owner.Stretch goal: all profile mutations pass through the profile store.
The text was updated successfully, but these errors were encountered: