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

Dungeon crawler update #530

Merged
merged 2 commits into from
Mar 20, 2025
Merged

Dungeon crawler update #530

merged 2 commits into from
Mar 20, 2025

Conversation

lillianhidet
Copy link
Contributor

Description

This PR improves the dungeon crawler tutorial, updates it to Godot 4.4, updates certain areas to match current best practice, and fixes minor correctness issues. Primarily:

  • Updates project to Godot 4.4 compatibility
  • Improves how weapon animation is handled, in a way that makes the tutorial easier to follow by compacting a lot of steps onto a single node
  • Improves how UI is handled by introducing singleton nodes, which also removes the previously bad way of accessing the UI node
  • Fixes a bug in which enemies would only damage the player upon entering the player hitbox, but would not continue to damage the player if they remained in contact with it.
  • Many minor correctness changes

Second version of this PR as the initial version was done without first pulling from main... Which is also why this unfortunately has no commit history.

Copy link

cloudflare-workers-and-pages bot commented Mar 20, 2025

Deploying wiki with  Cloudflare Pages  Cloudflare Pages

Latest commit: d486fb7
Status: ✅  Deploy successful!
Preview URL: https://c974755f.wiki-752.pages.dev
Branch Preview URL: https://dungeon-crawler-tutorial-upd.wiki-752.pages.dev

View logs

@lillianhidet lillianhidet enabled auto-merge March 20, 2025 01:41
@lillianhidet lillianhidet added this pull request to the merge queue Mar 20, 2025
Merged via the queue into main with commit 61d0b5e Mar 20, 2025
3 of 6 checks passed
@lillianhidet lillianhidet deleted the dungeon-crawler-tutorial-update branch March 20, 2025 01:44
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