qb-core icon indicating copy to clipboard operation
qb-core copied to clipboard

💱 Added money transfer functions

Open Cocodrulo opened this issue 10 months ago • 7 comments

Description

Implementation of the suggestion [SUGGESTION] Implement monetary transactions as a function. There have been added to two new functions. One in functions (QBCore.Functions.TransferMoney) and other one on player (Player.Functions.TansferTo), both aproach the same objective, trasnfering money betwwen two players if both are online, online one is online or both are offline.

Checklist

  • [ ] I have personally loaded this code into an updated qbcore project and checked all of its functionality.
  • [x] My code fits the style guidelines.
  • [x] My PR fits the contribution guidelines.

Cocodrulo avatar Apr 18 '24 12:04 Cocodrulo

Black money is not in the core, but crypto is, and you didn't put it in ?

MonsieurBibo avatar Apr 18 '24 20:04 MonsieurBibo

Good point, but actually that information is only commented information, I mean is not really whitelisting those monetypes it is only info not actual code.

Anyway I'll fixed it tomorrow

Cocodrulo avatar Apr 18 '24 21:04 Cocodrulo

Fixed the annotation and also renamed the Player.Functions.TransferTo to Player.Functions.TransferMoneyTo to make it self explainable.

Cocodrulo avatar Apr 19 '24 10:04 Cocodrulo

Changed those things that you commented, you were right in all of them jajaja

Cocodrulo avatar Jul 20 '24 20:07 Cocodrulo

Changed those things that you commented, you were right in all of them jajaja

Will check that soon and let you know if I find something else

Scartane avatar Jul 20 '24 20:07 Scartane

Sorry I added changes by mistake, changes reverted

Cocodrulo avatar Jul 21 '24 16:07 Cocodrulo

Jus added logs and Money Events, a bit chunky the blocks of code but I don't know how to make it smaller.

Cocodrulo avatar Jul 26 '24 16:07 Cocodrulo