Prodigy-Hacking / ProdigyMathGameHacking

Prodigy Hacking: Hacking for the right reasons. | Questions? Ask on our Discord. https://discord.gg/XQDfbfq
https://prodigyhacking.com
Other
17 stars 10 forks source link

give 1e7 gold anyway #1195

Closed donaldli2020 closed 2 years ago

donaldli2020 commented 2 years ago

Description

Screenshots

00100000 commented 2 years ago

This is a bad way to design things. The user didn't want 10,000,000 gold, don't set their gold to that amount. Also, prodigy's gold limit was addressed by #1184, please do not make further pull requests regarding it.

PatheticMustan commented 2 years ago

the user might not have wanted 10m gold, but they definitely wanted a lot of it. I think it was a reasonable change but unnecessary.

00100000 commented 2 years ago

The issue is, the UI doesn't have gold count on display, and the user might not read the whole error. I know that I certainly don't carefully read every pop-up. Giving them 10,000,000 gold without them being fully aware could lead to them battling, getting the gold, and getting a 418.

PatheticMustan commented 2 years ago

If we limit the gold to 9m this could be avoided entirely...

00100000 commented 2 years ago

I don't think we should set a hard limit like the one I set at 10mil, but it would probably be good to add a warning if users set their gold above 9mil.