Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

Multi Player Sitting #122

Closed
wants to merge 12 commits into from
Closed

Multi Player Sitting #122

wants to merge 12 commits into from

Conversation

vabene1111
Copy link

@vabene1111 vabene1111 commented Apr 13, 2018

When merged this pull request will:

  • allows multiple players to sit on one object (bench)
  • adds Land_WoodenLog_F to sittable objects

With this PR every object can be configured to support multiple players sitting on it.
I have tried to integrate this as clean as possible and it is extendable for other modders to add multi sitting in their objects.
This feature hast been tested both locally and on a dedicated server.

I tried to create this feature following all of ACE's style regulations, but in case i missed please let me know and i will fix it.

@jonpas jonpas added this to the 3.4.0 milestone Apr 13, 2018
@jonpas jonpas added the WIP label Apr 14, 2018
@jonpas
Copy link
Member

jonpas commented Apr 14, 2018

Discussing potential better ways of handling this on Slack with @vabene1111.

@stale
Copy link

stale bot commented Nov 10, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Nov 10, 2018
@jonpas jonpas removed the wontfix label Nov 10, 2018
@PabstMirror PabstMirror modified the milestones: 3.4.0, Ongoing Nov 15, 2018
@jonpas
Copy link
Member

jonpas commented Nov 18, 2018

Reworked/improved in #144

@jonpas jonpas closed this Nov 18, 2018
jonpas added a commit that referenced this pull request Nov 23, 2018
**When merged this pull request will:**
- Add everything in #122 in a better, more integrated way
- Add Arm Chair and fix inheritance on benches
- Improve variable naming
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants