Shop menu and Money System

[iquelli]

Closes #43, #108

[TiagoFar78]

I liked that you created menus package, but I think Chest and Vault classes should stay on prisonbuilding package. They are more of a block than a menu. They just happen to have a menu to represent them.

[iquelli]

I liked that you created menus package, but I think Chest and Vault classes should stay on prisonbuilding package. They are more of a block than a menu. They just happen to have a menu to represent them.

But the part that's implemented in the class is the menu, not exactly anything related to where they are in the prison.

[iquelli]

Think we could name the classes VaultMenu instead for example? Not sure. But what about clickable and ClickReturnAction, can I leave them in menu package?

[TiagoFar78]

But what about clickable and ClickReturnAction, can I leave them in menu package?

Yes, and let Shop too.

Think we could name the classes VaultMenu instead for example? Not sure.

I don't think it is a good idea to change Vault and Chests name. Maybe the best ideia is to create a new class named VaultMenu and doing menu related things there. And in this case Vault would have a VaultMenu.

However, this is not that relevant and we don't have to worry that much about it since it is just a question of where to place classes. You can leave the code as it is right now.

[TiagoFar78]

good job!