Bank API should support discord.Object when getting Balance

[TrustyJAID]

Feature request

Select the type of feature you are requesting:

• [ ] Cog
• [ ] Command
• [x] API functionality

Describe your requested feature

Cores Bank API should accept discord.Object since we cannot always ensure that it's safe to acquire a full user/member object when accessing things as trivial as an economy balance.

[Stonedestroyer]

I will be doing this in the upcoming days.

[goettner]

I'd like to take a look at this issue, but I'm not totally sure how to go about accepting a discord.Object as a parameter for balance() in economy.py instead of a Member. Would it be a decent solution to extract User.id, and pass this along to bank.get_balance() instead (which could possible be used in other calls, like transfer()), or am I missing something there? Thanks!

[Drapersniper]

The ideas is you'd accept either a member object, user object or discord object, which all 3 have the .id attribute.

Essentially extracting that attribute in the method it self when the id is needed (for config calls)

[goettner]

Okay, I understand, I'll take a crack at it. Thanks for the clarification