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 reward option #15

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

autoit4you
Copy link

How about the option to give the player a reward for winning the game?
Please consider it.

@DerpgonCz
Copy link
Contributor

How about support for multiple IDs, Data values and such?
Example: 3 grass, 5 spruce planks, 1 stone
2,3;4:1,5;1,1

@@ -3,11 +3,13 @@
import java.util.ArrayList;
import java.util.Date;
import java.util.HashMap;

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whitespace changes!

@autoit4you
Copy link
Author

You can now give more than one item

@Double0negative
Copy link
Owner

Please use ItemReader.read(string); to load items.

Look at https://github.com/Double0negative/Survival-Games/blob/master/src/main/resources/kits.yml for more info regarding the format.

@autoit4you
Copy link
Author

I think I am done with this pull request. Could you please consider this now?

@autoit4you
Copy link
Author

Bump

@ghost
Copy link

ghost commented Jun 16, 2013

Is this PR Gonna be added? :(

@PatchGuru
Copy link

I bump this to add it, I have talked to a few servers which believe adding a stable reward system for winning each game should be put in place. Something along the lines of a specific amount of currency is given depending on the number of players in a game. Which would be configurable.

@ArtificialTruth
Copy link

BUMP 👍

@wesley27
Copy link

Bump! This PR needs to be added...its an essential part of any hunger/survival games.

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.

7 participants