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

Updated to exiled 3 #17

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

Conversation

steven4547466
Copy link
Contributor

The SH and CiSpy dr brights uses is not the same as the public versions which is why I removed them from this branch before making this PR.

Also sorry about the massive file changes, my visual studio decided it would format it stupidly so I just reformatted files (which for some reason included package files). What I actually updated/added was:

  • Updated to Exiled 3 (obviously)
  • Changed AFKComponent
  • Added AFKData struct
  • Moved most of the logic out of the delay and instead took advantage of the Spawning and ChagingRole events which gives us access to change position and items and is more efficient than a delay. Unfortunately there is no ammo, hp or 079 data in these events so those are still delayed
  • Updated the drop item event (ItemDropped was removed)

I don't think this will need to be updated again for the full release of exiled 3, but if it does I'll PR those changes as well as we actively use it so why not.

@steven4547466
Copy link
Contributor Author

I should add this will NOT work on the current live versions of SL or even with exiled 2. If you do merge this you should ensure you add this in the release notes

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.

1 participant