Skip to content

Interaction - Add door events#9910

Merged
LinkIsGrim merged 6 commits intoacemod:masterfrom
ilbinek:interaction-add-door-events
Apr 3, 2024
Merged

Interaction - Add door events#9910
LinkIsGrim merged 6 commits intoacemod:masterfrom
ilbinek:interaction-add-door-events

Conversation

@ilbinek
Copy link
Copy Markdown
Contributor

@ilbinek ilbinek commented Apr 2, 2024

When merged this pull request will:

Given that all the important vars are made global just before this event is risen and it's all local, I don't think it's necessary to use arguments.

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

Comment thread addons/interaction/functions/fnc_openDoor.sqf Outdated
ilbinek and others added 2 commits April 2, 2024 23:15
Co-authored-by: Jouni Järvinen <rautamiekka@users.noreply.github.com>
@PabstMirror PabstMirror added this to the 3.17.1 milestone Apr 2, 2024
@PabstMirror PabstMirror added the kind/enhancement Release Notes: **IMPROVED:** label Apr 2, 2024
Copy link
Copy Markdown
Member

@LinkIsGrim LinkIsGrim left a comment

Choose a reason for hiding this comment

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

Documentation under Events Framework page, and possibly pass _house and _animations/_position (whichever one is the actual door) to event

ilbinek added 2 commits April 3, 2024 02:40
Adds the _house - object the door is in, _door - which doors are being openened, _animation - list of animation on this door
Comment thread addons/interaction/functions/fnc_openDoor.sqf Outdated
@LinkIsGrim LinkIsGrim merged commit 558849b into acemod:master Apr 3, 2024
@ilbinek ilbinek deleted the interaction-add-door-events branch April 3, 2024 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement Release Notes: **IMPROVED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants