Based on my interpretation of the relationships, it is wrong. I take it that "bridge" tables are meant to be junction tables. You're linking PK fields in junction tables to parent tables rather than pk in parent to fk in child, then between junction and parent/child. In one or more cases you have pk fields that are not joined to anything.
Aside from the relationships, did they not teach you to avoid using special characters in object names (save for underscore _ ) and to not store calculations in 99.9% of cases? They should have. Note that you cannot post db copies by cut and paste. You must follow the same process as how you got the images in your post (unless things have changed here). So if you meant to provide a db copy, most of the time it must be zipped and should be compacted first.
The more we hear silence, the more we begin to think about our value in this universe.
Paraphrase of Professor Brian Cox.