Brian Manifold
Brian Manifold
@jamilbk, yeah, apologies, I've been meaning to either mark this as a draft or just close it. Do you have a preference?
> @bmanifold Would this PR break your monitoring VM? > > Since there will be a headless Client for Windows in the near future, I'm trying to slowly remove the...
I spun this up on my local machine and did a `drop`, `create`, `migrate`, `seed` on the DB to make sure it was in a good state and then just...
@AndrewDryga, should we put this PR in draft for now?
@jamilbk, yep. @AndrewDryga, I'm wondering if fetching the first 25 is even necessary. If you think about `groups` as an example, we've already seen accounts where they have many more...
~I still need to finish up Actor Group Memberships, as well as potentially the create/update for Identities, but I figured since I was going to be out the next two...
Yeah, would definitely be nice to be able to turn on/off without having to redeploy. As far as the looks, no objections here. Just out of curiosity, did you try...
> Yeah the website banner is a strong primary, but is dismissible. I thought it might be too distracting. I can try with a lighter primary shade and/or make it...
> @bmanifold trying to get it tested in staging, we can address PR review separately 👍 Sounds good
> Instead of using `JSONB` for actor_emails above, it makes more sense to just have a dedicated table for this, `actor_emails`. That way our index will be fast and efficient...