Tale of Kingdoms: A new Conquest

Tale of Kingdoms: A new Conquest

49.7k Downloads

Master - [merged]

SamB440 opened this issue ยท 13 comments

commented

In GitLab by @KingCobra_200 on Nov 10, 2020, 14:21

Merges master -> master

Fully Implemented the Banker.
Checks for no coins. Message saying to slay monsters for coins.
Deposits and Withdraws with correct Exit message.

commented

In GitLab by @KingCobra_200 on Nov 10, 2020, 14:46

added 7 commits

Compare with previous version

commented

In GitLab by @KingCobra_200 on Nov 10, 2020, 15:16

added 3 commits

  • f72b840 - Merge branch 'master' of https://gitlab.com/SamB440/tale-of-kingdoms
  • de30b6c - Fixed banker methods.
  • da6a04a - Merge branch 'master' of https://gitlab.com/EduardoGarza00/tale-of-kingdoms

Compare with previous version

commented

Please don't use brackets etc like this. https://google.github.io/styleguide/javaguide.html

commented

Same as +44 to +48

commented

Use the local variable instead.

commented

Why?

commented

In GitLab by @KingCobra_200 on Nov 10, 2020, 16:03

Commented on src/main/java/net/islandearth/taleofkingdoms/client/gui/entity/BankerScreen.java line 88

changed this line in version 4 of the diff

commented

In GitLab by @KingCobra_200 on Nov 10, 2020, 16:03

Commented on src/main/java/net/islandearth/taleofkingdoms/client/gui/entity/BlacksmithScreen.java line 11

changed this line in version 4 of the diff

commented

In GitLab by @KingCobra_200 on Nov 10, 2020, 16:03

added 1 commit

Compare with previous version

commented

resolved all threads

commented

approved this merge request

commented

enabled an automatic merge when the pipeline for 8a6982d succeeds

commented

mentioned in commit a48cca4