Master - [merged]

[SamB440]

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.

[SamB440]

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

added 7 commits

• abd0ec1...950ddda - 6 commits from branch SamB440:master
• 57bc39c - Merge branch 'master' into 'master'

Compare with previous version

[SamB440]

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

[SamB440]

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

[SamB440]

Same as +44 to +48

[SamB440]

Use the local variable instead.

[SamB440]

Why?

[SamB440]

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

[SamB440]

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

[SamB440]

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

added 1 commit

• 8a6982d - Fixed issues.

Compare with previous version

[SamB440]

resolved all threads

[SamB440]

approved this merge request

[SamB440]

enabled an automatic merge when the pipeline for 8a6982d succeeds

[SamB440]

mentioned in commit a48cca4