Trey
Trey
Would you like to be unassigned from this issue @kryptokun?
 Open to all :)
The reason I added that to the usermanager is because that data is needed for EVERY page as it's in the topbar. I feel like that'll cause more issues taking...
- Needs @devenderbutani21 to have ran `black ./` - ~~Awaiting @TreyWW to fix tests on main branch~~
Hmm since we've moved to SES for emails we've not continued support for Sendgrid. Maybe I should do a revamp for emails and allow for either to be used?
I suggest next time running tests locally, there's been quite a few issues with this PR. https://docs.strelix.org/MyFinances/contributing/setup/#test-and-lint
Apologies I force pushed some stuff the other day and it broke a few bits. I'll revert it now
Apologies for the really delayed response. If you could fix up the bug that I addressed above that would be much appreciated.
> @TreyWW hmm basically it's all caused by `CustomUserMiddleware` > > ```py > > if request.user.is_authenticated: > > request.user = User.objects.get(pk=request.user.pk) > > ``` > > is causing 2nd query...
Our one also prefetches things to do with profile info. It makes sense to do it all in one query rather than a second query