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

Solved infinite fall issue. #188

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

martinbelcic
Copy link

@martinbelcic martinbelcic commented Jun 8, 2020

Add Checkpoint to Lv 1.

Please check if the PR fulfills these requirements:

  • The commit message follows our guidelines.
  • For bug fixes and features:
    • You tested the changes.
    • You updated the docs or changelog.

Related issue (if applicable): #187 #185

What kind of change does this PR introduce?
Added a Checkpoint on Level 1, which prevents the game from crashing if you hook multiple times to a single target. I've also created a new Death Zone scene which kills the player instantly to avoid falling endlessly

Add Checkpoint to Lv 1.
@NathanLovato
Copy link
Contributor

Thanks for the contribution, but in this project, the player just shouldn't be able to fall in pits. If there's a missing wall in a given level, we should add it instead.

@NathanLovato
Copy link
Contributor

Ah sorry, I had mistaken the project. We have so many repositories at the moment...

Yes so at the moment, the game is not finished and the player can fall indefinitely. However, I don't know if this is a good fix - Ideally, we shouldn't have to place the death colliders manually in every level - it's too error-prone. Also, your code doesn't match our GDScript guidelines. Please check them out to contribute: https://www.gdquest.com/docs/guidelines/best-practices/godot-gdscript/

@NathanLovato NathanLovato reopened this Jun 8, 2020
@NathanLovato
Copy link
Contributor

I have to think about the best way to address that. One level-agnostic way to do that is to have a counter on the player that keeps track of the fall height, and they die past a certain value.

Another option is to have a general death height variable on the Level or player that causes the player to die, maybe with some visual indicating the limits of the level both in the editor and in the game.

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.

None yet

2 participants