Skip to content

Conversation

@CyberL1
Copy link
Contributor

@CyberL1 CyberL1 commented Dec 5, 2025

No description provided.

@MathMan05
Copy link
Owner

this should probally be a check for false-y, could you change it to that rq? (so it doesn't break previous Fermi saves)

@CyberL1
Copy link
Contributor Author

CyberL1 commented Dec 5, 2025

this should probally be a check for false-y,

It shouldn't
obraz

@MathMan05
Copy link
Owner

The thing is is that Fermi due to old server behavior set id to 0, so this is a breaking change, it should just be a false-y check, as id shouldn't be 0 anyways.

@CyberL1
Copy link
Contributor Author

CyberL1 commented Dec 5, 2025

it should just be a false-y check, as id shouldn't be 0 anyways.

But wait, !!null, or !!0 return false instead of null. I don't think that'd work?

@MathMan05
Copy link
Owner

folder.id === null -> !folder.id

@MathMan05
Copy link
Owner

alrighty, looks good!

@MathMan05 MathMan05 merged commit 0556a35 into MathMan05:main Dec 5, 2025
1 check passed
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.

2 participants