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

feat: RPG-129 conditional actions #67

Merged
merged 38 commits into from
Jun 3, 2022
Merged

Conversation

kkafar
Copy link
Collaborator

@kkafar kkafar commented Jun 1, 2022

Description

State for now:

You can specify item-required condition for any type of action.

As for now the Inventory is still not implemented (see #64), so I need to defer specific implementation.

You can see for your self that the mechanism works by manually modifying the value here and trying to teleport using the bunny in starting location (when this value is set to false, teleport won't work).

Checklist

  • Included code to test these changes
  • Updated Jira

@kkafar kkafar changed the title chore: init feat: conditional actions Jun 1, 2022
@kkafar kkafar changed the base branch from master to @kkafar/depend-on-kkafara-result June 1, 2022 20:19
@kkafar kkafar changed the base branch from @kkafar/depend-on-kkafara-result to master June 2, 2022 07:15
@kkafar kkafar requested review from co012, mhawryluk and ksiek127 and removed request for co012, mhawryluk and ksiek127 June 3, 2022 20:20
@kkafar kkafar marked this pull request as ready for review June 3, 2022 20:24
@kkafar kkafar changed the title feat: conditional actions feat: RPG-129 conditional actions Jun 3, 2022
Copy link
Collaborator

@co012 co012 left a comment

Choose a reason for hiding this comment

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

LGTM

@kkafar kkafar merged commit ba49bef into master Jun 3, 2022
@kkafar kkafar deleted the @kkafar/conditional-actions branch June 3, 2022 21:00
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