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

Feuer fixes #30

Merged
merged 35 commits into from
Feb 23, 2024
Merged

Feuer fixes #30

merged 35 commits into from
Feb 23, 2024

Conversation

FireInstall
Copy link

reopen of #23 - I have renamed the branch since I have multible dev banches open

dokumentation
Added GreenSurvivors as authors
renamed blockRecorder package to just recorder to reflect that entities get recorded as well. Also, it makes codacy happy.
some minor fixes
added GameEnded trigger that allways fires if a game has come to an end (not only if it is in its endphase)

FireInstall and others added 30 commits December 15, 2023 03:16
dokumentation
renamed blockRecorder package to just recorder to reflect that entities get recorded as well. Also, it makes codacy happy.
Fix Volume calculation of regen regions being one block of in each axis
I have one question: why?
fix waxed signs not receiving an update
… ended and not only if the GameEndPhase was reached.
… null

  * Teleports the spectating players to the end location if a quit location is not set.
  Fixes Spectators getting stuck in Minigame if no endpostion was set.
fix air test for treasures
fix hardcoded number of inventory slots for treasures (allowing other containers getting used in future if someone decides to make this part configurable)
added description, so the plugin doesn't crash in case the outputCMDToFile was turned on
@FireInstall FireInstall requested a review from joshi1999 January 24, 2024 10:44
In rare occasions a user can be a spectator as well as a player (however that happened)
This may still have a desync problem with the client where menu items are invisible.
This, however is a client problem and even player.updateInventory will not help, as it get called anyway if the event gets cancelled.
@FireInstall
Copy link
Author

@FireInstall FireInstall self-assigned this Feb 10, 2024
@FireInstall FireInstall added the bug Something isn't working label Feb 10, 2024
Copy link

@joshi1999 joshi1999 left a comment

Choose a reason for hiding this comment

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

All in all this is looking very good ^^

Copy link

@joshi1999 joshi1999 left a comment

Choose a reason for hiding this comment

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

Nice, well done ^^

@FireInstall FireInstall merged commit 0b8d6cd into dev Feb 23, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants