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

a lot of changes fixed a lot of bug fixes and added a lot of todo's and many more #389

Draft
wants to merge 15 commits into
base: dev
Choose a base branch
from

Conversation

hiptodude2
Copy link

i seriously don't have a description i contributed and I'm happy to do so I wasted most of my time on this just to see people not get mad at one pixel ruining their whole gameplay (lol)

@mestiez
Copy link
Contributor

mestiez commented Dec 8, 2024

is there a difference between this and #388??
EDIT: i see, this one is trying to merge to dev as opposed to the other one trying to merge into main (not allowed)

@mestiez
Copy link
Contributor

mestiez commented Dec 8, 2024

can you provide a comprehensive overview of what the intended changes are?

Scene.Type is always Scene so that makes no sense
The rest of the data is already included in the scene dump
@mestiez
Copy link
Contributor

mestiez commented Dec 9, 2024

uhh could you also explain what exactly is going on in d36ce74

@mestiez mestiez added the question Further information is requested label Dec 9, 2024
@mestiez mestiez marked this pull request as draft December 11, 2024 13:23
@mestiez
Copy link
Contributor

mestiez commented Dec 11, 2024

Marked as draft until my questions are answered :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants