human-essentials #3764: Fix profile attachments saving

Tried this out and all the tests are still passing when I take the reload out, but attachments are saving again. Not sure why it was in there in the first place. Did some desultory manual testing as well and I couldn't find any problems.

Jul 28 '23 20:07 dorner

@cielf not sure what happened but I tried again and it looks like I was missing one more reload - reloading inside a transaction looks like it blows away the attachment changes even if it's already changed. Try now!

Aug 04 '23 20:08 dorner

@dorner Hrm. On my local, If you upload a file, and have counties not adding up to 100, and hit save, you, of course, get an error (as you should), but there is a link for the file, and if you click on it, you get one of those unpleasant "This localhost page can’t be found" errors. I wouldn't be surprised to learn that was always the case, of course. We probably haven't tested these in combo much. (This was with the determination letter)

Aug 04 '23 23:08 cielf

@cielf played around and it looks like this only happens on the edit screen itself when there was an error. If you refresh the edit page, it shows you the original attachment (the new one wasn't saved, which is correct) and it's fine. Not sure it's worth it to chase this bug.

Aug 11 '23 20:08 dorner

So... if I understand correctly, we would be in a better place than we originally were if we put this in without "chasing this bug". Given that, I say let's have @awwaiid take a look, then move on.

Aug 16 '23 20:08 cielf

@awwaiid Can you take a look at this one, please?

Feb 27 '24 15:02 cielf

@dorner: Your PR #3764: Fix profile attachments saving is part of today's Human Essentials production release: 2024.03.17. Thank you very much for your contribution!

Mar 17 '24 14:03 github-actions[bot]