Skip to content

make animationSpeed decalregistry handler accept floats#1097

Merged
microlith57 merged 1 commit intoEverestAPI:devfrom
microlith57:float-animationspeed
Mar 30, 2026
Merged

make animationSpeed decalregistry handler accept floats#1097
microlith57 merged 1 commit intoEverestAPI:devfrom
microlith57:float-animationspeed

Conversation

@microlith57
Copy link
Copy Markdown
Member

the underlying field is a float, so the handler should also accept floats

also fixed some whitespace

@maddie480-bot maddie480-bot added the 1: review needed This PR needs 2 approvals to be merged (bot-managed) label Mar 25, 2026
@microlith57 microlith57 added the 2: testing needed This PR needs to be tested on one or more platforms label Mar 30, 2026
Copy link
Copy Markdown
Member

@SnipUndercover SnipUndercover left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Considering it compiles I don't think testing before dev is necessary. It's just a type change that we can easily roll back if for whatever reason it combusts.

@microlith57 microlith57 removed the 2: testing needed This PR needs to be tested on one or more platforms label Mar 30, 2026
@microlith57 microlith57 merged commit e814207 into EverestAPI:dev Mar 30, 2026
3 checks passed
@microlith57 microlith57 deleted the float-animationspeed branch March 30, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1: review needed This PR needs 2 approvals to be merged (bot-managed)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants