Skip to content

fix #5026 issue and refactor ModelScene.ts#5101

Merged
diegoteran merged 4 commits intogoogle:masterfrom
mohammadbaghaei:fix-reverse-animation-play
Feb 27, 2026
Merged

fix #5026 issue and refactor ModelScene.ts#5101
diegoteran merged 4 commits intogoogle:masterfrom
mohammadbaghaei:fix-reverse-animation-play

Conversation

@mohammadbaghaei
Copy link
Contributor

No description provided.

Copy link
Collaborator

@diegoteran diegoteran left a comment

Choose a reason for hiding this comment

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

Thank you, looks good. would just appreciate the warnings to stay.

@diegoteran
Copy link
Collaborator

@mohammadbaghaei thanks for the contribution! I'll merge it soon.
Can you check this please?

If you are not currently covered under a CLA, please visit https://cla.developers.google.com/. Once you've signed, follow the "New Contributors" link at the bottom of this page to update this check.

@mohammadbaghaei
Copy link
Contributor Author

image

It seems correct, I don't know why it gives a validation error.

@diegoteran
Copy link
Collaborator

It seems correct, I don't know why it gives a validation error.

It shows two different users adding commits when it is only you. They both have the same email but one user has your account username and the other doesn't.

image

There might be a key difference in how you are pushing the commit.

@mohammadbaghaei mohammadbaghaei force-pushed the fix-reverse-animation-play branch from 95d58eb to 5195a36 Compare February 27, 2026 20:25
Copy link
Collaborator

@diegoteran diegoteran left a comment

Choose a reason for hiding this comment

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

thank you so much!

Next time please use the clang-format, I'll fix it this time.

@diegoteran diegoteran merged commit 305e07c into google:master Feb 27, 2026
3 checks 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