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

Implement multi-use items. #420

Merged
merged 1 commit into from Feb 20, 2015
Merged

Implement multi-use items. #420

merged 1 commit into from Feb 20, 2015

Conversation

@scurest
Copy link
Contributor

@scurest scurest commented Feb 18, 2015

When using an item, only one "use" should be used up now.

A quick test in RM2K showed

  • the number of uses resets when an item is removed
  • the number of uses stays the same when an item is added, even if the player already had x99 of the item

Also, for the same reason as #399, using a special item in battle consumes a number of uses equal to the number of targets, currently.

@Ghabry
Copy link
Member

@Ghabry Ghabry commented Feb 18, 2015

Jenkins: ok to test

@fdelapena
Copy link
Contributor

@fdelapena fdelapena commented Feb 20, 2015

Thanks @scurest! In order to get automatic Jenkins pull requests building for automated tests in multiple platforms, please accept the "Player team" invitation from GitHub to get your user whitelisted in the Jenkins Continuous Integration service. This will prevent the need of the "okay to test" comment for every opened pull request.

Ghabry added a commit that referenced this pull request Feb 20, 2015
Implement multi-use items.
@Ghabry Ghabry merged commit cd454eb into EasyRPG:master Feb 20, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@fdelapena
default Merged build finished.
Details
@scurest scurest deleted the scurest:itemuses branch Feb 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants