Maybe store `acct` with the User
Originally the User struct looked like this:
And it served only to link these two data points. I became convinced it would be better to make the
ap_id the unique identifier, so I switched out
Now I'm starting to regret it, because getting donor info about a user looks like this:
When ideally it could look like this:
But I still have mixed feelings. I think I want to store both in the database, but we'll have to be sure both are present.