search

thecsw / memeinvestor_bot

This bot can help you invest in memes and make a fortune out of it!
https://meme.market
GNU General Public License v2.0
135 stars 34 forks source link

Fixing some silly mistakes with the cosmetic firm officer values #443

Closed Keanu73 closed 5 years ago

Keanu73 commented 5 years ago

Alright, so now I see a couple of mistakes in my code which may have lead to craziness like this: {"id":158,"name":"EMPYREAN","balance":787098244161521,"size":79,"execs":11,"assocs":20,"ceo":"**JonathanTheZero**","coo":"**JonathanTheZero**","cfo":"**JonathanTheZero**","tax":5,"rank":4,"private":true,"last_payout":1562364233}

Including accidentally using the person who ran the command's name to set as the firm's CFO or COO value INSTEAD of the actual target....

I also added a check to check if the value is still 0 (remember, when we first ran the SQL script values that were 0 were never changed so we need to make sure they are supported too).

thecsw commented 5 years ago

Ultimately, this is my fault. Thanks for looking into it! It will get reviewed and hopefully merged tonight!