Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Vault support #253

Open
wants to merge 1 commit into
base: indev
Choose a base branch
from

Conversation

Reldeam
Copy link

@Reldeam Reldeam commented Feb 24, 2022

I created a EconomyComponent that extends a QuartzComponent but I'm
unfortunately not fully across the QuartzLib pattern so I will probably
need some guidance there.

The base functionality should be included to withdraw money from
player's accounts if they enable economy in the config and set costs.

What I would like to include next is notifications to the player so they
know how much has been removed or if an error occurred. I would also like
to include a confirm/deny option which tells the player how much the
operation is going to cost them before they do it.

I'm uncertain of the best way to go about telling the player how much
a copy of a poster is going to cost before they take it.

Also, I know you are a predominantly French team, so I am sorry that I
only know English. Any assistance is greatly appreciated :)

I created a EconomyComponent that extends a QuartzComponent but I'm
unfortunately not fully across the QuartzLib pattern so I will probably
need some guidance there.

The base functionality should be included to withdraw money from
player's accounts if they enable economy and set costs.

What I would like to include next is notifications to the player so they
know how much has been removed or if an error occured. I would also like
to include a confirm/deny option which tells the player how much the
operation is going to cost them before they do it.

I'm uncertain of the best way to go about telling the player how much
a copy of a poster is going to cost before they take it.
@AmauryCarrade
Copy link
Member

Thanks! We'll nook into this as soon as possible (busy days, but we'll do our best!) :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants