Revert transform code and fix player.tscn and fix game_entry #18

Merged
noahg merged 7 commits from revert-transform into dev 2026-03-07 19:29:53 +00:00
Member
No description provided.
@ -75,2 +75,4 @@
func scene_changed():
var root := get_tree().edited_scene_root
if not root:
Author
Member

This was causing crashes in editor when trying to open the project

This was causing crashes in editor when trying to open the project
@ -1,6 +1,6 @@
extends CanvasLayer
const MUSEUM_1 = preload('uid://crv6kgdf05l0p')
const MUSEUM_1 := preload("uid://crv6kgdf05l0p")
Author
Member

This was causing crashes when trying to run the game without first specifying a run config

This was causing crashes when trying to run the game without first specifying a run config
Member

Not sure where but the tanukui also lost its materials along the way. 😢

image

Not sure where but the tanukui also lost its materials along the way. :cry: ![image](/attachments/2bdc0232-4872-4943-8975-0500195060de)
Owner

@devonereynolds wrote in #18 (comment):

Not sure where but the tanukui also lost its materials along the way. 😢

image

This was my doing, it's fine.
Cleaning up the way the tanuki gets imported. I'm going to use materials instead of vertex colors. So in the meantime, he's just gray.

@devonereynolds wrote in https://git.bugjam.dev/BUGJam/pounce-game/pulls/18#issuecomment-351: > Not sure where but the tanukui also lost its materials along the way. :cry: > > [![image](/attachments/2bdc0232-4872-4943-8975-0500195060de)](/BUGJam/pounce-game/attachments/2bdc0232-4872-4943-8975-0500195060de) This was my doing, it's fine. Cleaning up the way the tanuki gets imported. I'm going to use materials instead of vertex colors. So in the meantime, he's just gray.
Author
Member

Not sure where but the tanukui also lost its materials along the way. 😢

I think that is from recent changes in swapping the model over to the rigged one? There have been a lot of changes for that model specifically lately.

> Not sure where but the tanukui also lost its materials along the way. :cry: I think that is from recent changes in swapping the model over to the rigged one? There have been a lot of changes for that model specifically lately.
noahg force-pushed revert-transform from ffce713777 to a827460686 2026-03-07 19:25:55 +00:00 Compare
Member

This PR is close to fixing the recent batch of changes. There are more broken commits that also need to be reverted.
Somehow while rebasing the branch @Alexis_Aurora ended up making broken copies of other people's commits.

The three commits that are broken copies of previous commits are:
Zoom camera in/out with mouse wheel / a2f7368c27 is a broken copy of 5b33d7d410.
Free-moving overhead camera / e1978eeb48 is a broken copy of a48dcc97c6.
added worldenvironment change to nate's test scene / 3fc57602b4 is a broken copy of 5fc3a86396.

I'm certain this is the case because these incorrectly rebased commits show another contributor as the author and @Alexis_Aurora as the committer.
Also, I remember these broken copies of commits were giving me trouble while trying to help get the original PR #10 for these changes fixed so it could be merged.

In summary, we also need to revert the commits above in reverse chronological order to avoid merge conflicts and fully back out these breaking changes. In the interest of time I went ahead and took care of doing this and force pushed this branch.

This PR is close to fixing the recent batch of changes. There are more broken commits that also need to be reverted. Somehow while rebasing the branch @Alexis_Aurora ended up making broken copies of other people's commits. The three commits that are broken copies of previous commits are: `Zoom camera in/out with mouse wheel` / a2f7368c27bcc6bb8a082a22b7ff95c05770d6bd is a broken copy of 5b33d7d4108820db0457a72e2f60049cf3256d10. `Free-moving overhead camera` / e1978eeb4811bb15909d714f315a6d4888e0fa44 is a broken copy of a48dcc97c683d8eeb19fcec720b4ff2ea66c8113. `added worldenvironment change to nate's test scene` / 3fc57602b4ffb368cc7ecf80287777c71e18a520 is a broken copy of 5fc3a863962ac44706ef2d3f97e593d3fd2e3437. I'm certain this is the case because these incorrectly rebased commits show another contributor as the author and @Alexis_Aurora as the committer. Also, I remember these broken copies of commits were giving me trouble while trying to help get the original [PR #10 ](https://git.bugjam.dev/BUGJam/pounce-game/pulls/10) for these changes fixed so it could be merged. In summary, we also need to revert the commits above in reverse chronological order to avoid merge conflicts and fully back out these breaking changes. In the interest of time I went ahead and took care of doing this and force pushed this branch.
noahg requested review from noahg 2026-03-07 19:28:46 +00:00
noahg approved these changes 2026-03-07 19:28:59 +00:00
noahg deleted branch revert-transform 2026-03-07 19:29:54 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
BUGJam/pounce-game!18
No description provided.