* [Economy][Bank] redo branch
* WIP
WIP
* Implement all current bank commands API calls
* Set dunder all and put into bot
* make core change to economy
* Add is_global method to bank
WIP
* Add extra bank API commands
* Update bank UI
Update some imports
Remove bank UI errors file
Typing thing
* Update bank get_global_accounts and touch up economy some more
Do some more economy updates
* Remove bank from bot
* Another passing test
FINALLY
* Fixy type things
Last fixes for now
Fix arg to toggle global
RJM
Invalid bid amount handler
cooldown msg
currency name fix
Fix fun bug
ANother bug
And payday limit
* PEP8 stuff
* Docstring change
* Fix this thing
* [Economy][Bank] redo branch
* [Economy][Bank] modify guild owner or bot owner check, add admin or bot owner check for global vs local bank
* [Economy] apply admin or bot owner check to [p]economyset
* Make some public things private
* [Economy] lots of refactoring for conditional permission checks and guild checks + supporting global economy
* And working stuff
* Fix Kowlin's bug
* Fix slot bugs
After giving it further thought, I don't believe co-owners really need to be a command, as it is unlikely that co-owners will need to be changed often enough.
Furthermore, this gives a further layer of protection to people who like to gloss over warnings.
Since these helpers will mostly be used by packages, it's important to make sure that the data they create stays contained inside them. This also brings the additional benefit of being able to manipulate data inside a package without knowing the name of the package's folder itself
After some considerations, while the flusher was an interesting experiment, it doesn't seem to be reliable enough. It's been removed in favor of the helpers autosaving on any change in a threadsafe way.
Will
palmtree5
Kowlin
Twentysix