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

Bug: Lego Dice item does not complete rolling animation #237

Closed
ckawell opened this issue Dec 15, 2021 · 7 comments
Closed

Bug: Lego Dice item does not complete rolling animation #237

ckawell opened this issue Dec 15, 2021 · 7 comments

Comments

@ckawell
Copy link
Contributor

ckawell commented Dec 15, 2021

NOTE: This Bug is addressed in this PR. This issue report is to document the issue.

When the Lego Dice (12513) is rolled, the rolling animation appears in the sky, then disappears without completing the roll. This is because the logic for the roll is not implemented server side.

@NealSpellman
Copy link
Member

You've failed to follow the issue template. Closing this and reopening with proper template.

@jakawell
Copy link

@NealSpellman The Bug template (and all templates other than "Feature") are broken. When folks try to open a non-feature issue they're forced to create a blank issue with no template.

@NealSpellman
Copy link
Member

@NealSpellman The Bug template (and all templates other than "Feature") are broken. When folks try to open a non-feature issue they're forced to create a blank issue with no template.

Yep, been reported to the member who wrote those but in the meantime you could copy from currently existing bug reports or use the template as it exists in the files to set it up. At the very least, some semblance of the same format would've been appreciated.

I've opened a new issue following the format; #238

@ckawell
Copy link
Contributor Author

ckawell commented Dec 15, 2021

Just to clarify @NealSpellman, I was the one who opened the issue and PR, not jakawell.
Because the templates were bugged I just wanted to get something out there real quick to provide more info for the fix in my PR.
Thanks for the comment and I'll copy/paste in the future, but brace yourself for more people not following the template until it exists again :)

@jakawell
Copy link

Too many Kawell's lol 😉

@NealSpellman
Copy link
Member

Just to clarify @NealSpellman, I was the one who opened the issue and PR, not jakawell. Because the templates were bugged I just wanted to get something out there real quick to provide more info for the fix in my PR. Thanks for the comment and I'll copy/paste in the future, but brace yourself for more people not following the template until it exists again :)

You both really have to have usernames with -kawell, huh? :P My bad @jakawell, I autopiloted my response seeing the end of your username.

And, even with templates bugged (as of writing this, they are fixed), we still expect users to follow the format. We're trying to keep issues properly formatted and including every detail our template asks for, as they help cut down on time other devs have to spend looking at it / researching / guessing missing details. This case wasn't super important, at least!

@ckawell
Copy link
Contributor Author

ckawell commented Dec 15, 2021

Glad to see the fix rolled out so quickly, thanks!

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

No branches or pull requests

3 participants