CustomDCBot icon indicating copy to clipboard operation
CustomDCBot copied to clipboard

Updated the economy-system

Open jateute opened this issue 3 years ago • 16 comments

Added user-feedback and fixed a few bugs

jateute avatar Jan 02 '23 19:01 jateute

Thanks so much for your contribution - to make review easier, could you provide us with a list of (User facing) changes? Doesn't need to be detailed, but should include the most important changes.

SCDerox avatar Jan 02 '23 21:01 SCDerox

Changes:

  • Added ids for items
  • Added a select menu for the shop
  • Added a shop-message (overview over the shop; gets send in a channel and works like the leaderboard)

jateute avatar Jan 03 '23 10:01 jateute

Do you have any updates on this?

SCDerox avatar Feb 03 '23 08:02 SCDerox

Do you have any updates on this?

Unfortunately I hadn‘t any time to finish the fixes. I‘ll try to resolve the problems asap

jateute avatar Feb 10 '23 20:02 jateute

Do you have an update on this? :sweat_smile:

SCDerox avatar May 16 '23 09:05 SCDerox

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Aug 29 '23 12:08 CLAassistant

Also, when /economy crime fails, the money you loose is calculated and taken from your deposited money, instead of your bank money. A possible solution to this would be to take it from where the most money is (take money from bank if there is more, else take it from deposited money).

(Sadly the screenshot is only available in German) image

Translation: "You tried to steal a wallet but got caught. You lose 0 🪙."

ElBe-Plaq avatar Jan 01 '24 10:01 ElBe-Plaq

It really needs help...

image

Translation:

Balance of elbe#0

Wallet: null 🪙

Bank: null 🪙

All: NaN 🪙

ElBe-Plaq avatar Jan 09 '24 10:01 ElBe-Plaq

@jateute I have converted this PR to a draft to make it more clear that it isn't ready to be used and work in progress. You can convert it back once it's ready to use.

SCDerox avatar Jan 09 '24 11:01 SCDerox

@jateute I have converted this PR to a draft to make it more clear that it isn't ready to be used and work in progress. You can convert it back once it's ready to use.

Thx, I didn't thought of it :hAA:

jateute avatar Jan 09 '24 12:01 jateute

Also, when /economy crime fails, the money you loose is calculated and taken from your deposited money, instead of your bank money. A possible solution to this would be to take it from where the most money is (take money from bank if there is more, else take it from deposited money).

(Sadly the screenshot is only available in German) image

Translation: "You tried to steal a wallet but got caught. You lose 0 🪙."

It could also work by giving you debt, so you don't lose anything and don't have 0 coins, but rather -X coins.

ElBe-Plaq avatar Jan 21 '24 10:01 ElBe-Plaq

It really needs help...

image

Translation:

Balance of elbe#0

Wallet: null 🪙

Bank: null 🪙

All: NaN 🪙

How did you do that? I wasn't able to reproduce that bug. For me the user either doesn't exist or I get normal numerical values

jateute avatar Jun 25 '24 16:06 jateute

It really needs help...

image

Translation:

Balance of elbe#0

Wallet: null 🪙

Bank: null 🪙

All: NaN 🪙

How did you do that? I wasn't able to reproduce that bug. For me the user either doesn't exist or I get normal numerical values

I somehow was able to get negative money I think. Sadly the server this screenshot was taken on is deleted now, so I can't check exactly what I did.

ElBe-Plaq avatar Jun 25 '24 16:06 ElBe-Plaq

It really needs help... image Translation:

Balance of elbe#0

Wallet: null 🪙 Bank: null 🪙 All: NaN 🪙

How did you do that? I wasn't able to reproduce that bug. For me the user either doesn't exist or I get normal numerical values

I somehow was able to get negative money I think. Sadly the server this screenshot was taken on is deleted now, so I can't check exactly what I did.

image For me it also works with negative numbers :) Maybe I somehow fixed it

jateute avatar Jun 25 '24 17:06 jateute

Hi, I'm just checking in, it seems like some conversations were resolved without any comment / change, could you please check again for me (I assumed that you were going to commit more, but it seems like you requested a review and only fixed certain typos)? Thanks (:

SCDerox avatar Aug 29 '24 09:08 SCDerox

~~It seems like I forgot to push the changes. I‘ll check on it later, when I‘m back at the PC ^^~~

Edit: I did :hAA: Now the changes are actually pushed

jateute avatar Aug 29 '24 09:08 jateute