search

159356-1702-Extramural / capstone

Capstone Project
Mozilla Public License 2.0
5 stars 5 forks source link

Trading notices, Cancel Trade, Dice balancing #325

Closed akclint closed 7 years ago

akclint commented 7 years ago

Changed a few things with trading:

  1. When a trade is successful, the initiator gets a message saying the trade was successful. (#318)
  2. The trade popup menu is now always present (#313)

image

  1. When a Trade is started, the menu will change from "Trade with Player" to "Cancel My Trade" to allow the player to stop trading (#316)

image

  1. Adjusted dice roll to use all possible dice combinations before re-filling the array again. Should balance out both robber and other rolls.

  2. Fixed issue #217

timack commented 7 years ago

@akclint Great stuff! - I can look over this, this evening unless someone gets to it first.

flukejones commented 7 years ago

Small issue: The "Cancel Trade" button is still there after a trade was accepted.