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

when you log off on a ship and it moves, you should log on still in the ship, not in the middle of nowhere #486

Open
Rubydesic opened this issue Jun 1, 2023 · 5 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@Rubydesic
Copy link
Contributor

No description provided.

@Rubydesic Rubydesic added enhancement New feature or request good first issue Good for newcomers labels Jun 1, 2023
@m-doescode
Copy link

Hello!

I've tried my own hand at implementing this, if no one else has yet. Excuse any bad coding practices or formatting that doesn't align with the styleguide.

I'm a bit shy to open a PR because I want to make sure that everything is alright first, but anyway, here it is: https://github.com/m-doescode/Valkyrien-Skies-2-fork/tree/1.20.1/feature/teleport-reconnected-players

@m-doescode
Copy link

Of course there are still currently some redundant print statements that I'll clean up if this solution is accepted, but this is what I have so far.

@xiewuzhiying
Copy link
Contributor

There is already a similar mod Valkyrien Relogs.
But I think your solution is better because it doesn't add a new block.👍

@m-doescode
Copy link

Thanks! Should I open a PR for the fix?

@thetheaplant
Copy link

Absolutely.

ewoudje pushed a commit that referenced this issue Aug 15, 2024
… still in the ship, not in the middle of nowhere (#937)

* Added basic implementation of teleporting disconnected players back onto ships

* Removed print statements

* Extracted MAX_ONBOARD_TICKS from magic number

* Moved MAX_ONBOARD_TICKS to a config option, and added the ability for reconnected player teleportation to be disabled

---------

Authored-by: m-doescode <[email protected]>
xiewuzhiying pushed a commit to xiewuzhiying/Valkyrien-Skies-2 that referenced this issue Aug 28, 2024
… should log on still in the ship, not in the middle of nowhere (ValkyrienSkies#937)

* Added basic implementation of teleporting disconnected players back onto ships

* Removed print statements

* Extracted MAX_ONBOARD_TICKS from magic number

* Moved MAX_ONBOARD_TICKS to a config option, and added the ability for reconnected player teleportation to be disabled

---------

Authored-by: m-doescode <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants