Skip to content
This repository has been archived by the owner on Oct 29, 2022. It is now read-only.

Add support for BedrockEconomy + Capital #468

Open
MrBlasyMSK opened this issue Mar 14, 2022 · 5 comments
Open

Add support for BedrockEconomy + Capital #468

MrBlasyMSK opened this issue Mar 14, 2022 · 5 comments
Labels
Category: Feature Request This feature may or may not be added in the future Status: TODO This will be done in the future

Comments

@MrBlasyMSK
Copy link
Contributor

I wanted to know if it is possible to add support for BedrockEconomy and Capital since that is what is now mostly being used for API 4 and they are going to be updated with improvements and bug fixes (not like EconomyAPI).

  • These plugins will be updated with constant improvements.
  • EconomyAPI has become outdated in many ways.
  • People are moving from EconomyAPI to BedrockEconomy (from cooldogede9) or Capital (from SOF3).
  • It would help many users to choose the option they would like so that they do not have to change their economy plugin.
@jasonw4331
Copy link
Owner

jasonw4331 commented Mar 14, 2022

I have support for Capital on the libasynql branch. It's still very work-in-progress. It's unlikely I will support both due to the different async architectures they use

@MrBlasyMSK
Copy link
Contributor Author

I have support for Capital on the libasynql branch. It's still very work-in-progress. unfortunately it's impossible to support both due to the different async architectures they use

Is there no way to add BedrockEconomy?
I say this because most plugins are using that or EconomyAPI, here are some examples:

LuckyPlayer by MintoD
CustomShopUI by AlexPads
ChestShop by Muqsit
etc...

And it would be annoying for most people to have to change a complete Economy plugin for a different one and in which there are almost no existing plugins that have Capital support.

@jasonw4331
Copy link
Owner

jasonw4331 commented Mar 14, 2022

The asynchronous architecture I'm using is based on php's Generator class, but BedrockEconomy's is based on closures and threads, which makes it difficult to synchronize the result to be passed back into the first async chain. It's probably possible, but ridiculously over complicated

@MrBlasyMSK
Copy link
Contributor Author

Well after all you are the one who makes the plugin so I don't want to pressure you to do anything, I just hope that in the future you plan to add it.

If you want to close this issue

@jasonw4331 jasonw4331 added Category: Feature Request This feature may or may not be added in the future Status: TODO This will be done in the future labels Jul 20, 2022
@BoyKage
Copy link

BoyKage commented Aug 8, 2022

It would be nice to add it to BedrockEconomy because I also use it, and it would be annoying if I had to change the server's Economy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Category: Feature Request This feature may or may not be added in the future Status: TODO This will be done in the future
Projects
None yet
Development

No branches or pull requests

3 participants