Noble requests -- address TODOs

Noble requests are working, but there is a bug:
- User buys a card, is eligible for 2 nobles
- action panel appears, the user clicks the noble they want
- client sends this info to the server
- server updates the game state and sends it back to the client
- client calls `nextTurn` again and awards the next noble to the player (this is the bug)

TODOs in last commit. address.
You're right we should probably move `noblesEarned`.

As far as turning Player into a real object on the client I'm not sure how to turn this into something actionable. Previously there were methods on `Player` that only worked on the server so I don't think it was a data object before. Client code just has the POJO at the moment.