Feedback from PJB

1. stop using EFCore `.Find`, this error clearly shows it's a poorly designed API that should be marked as obsolete
2. you need to use async more, there's a ton of cases where you're doing DB ops without async which is bad for perf (doesn't matter in your case but bad form)
3. fix the nullability on the other DbSets