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
fix(status): Make status module work even when the status is 0 #3750
Conversation
I can test on Linux but don't have access to a Windows machine ATM. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You only need to update the English version of the docs, the other languages are handled automatically by Crowdin. If you remove the changes to the non English docs the tests should pass.
I see. I'll do it soon thanks |
It's complaining that I fixed indentation? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
Thank you for your contribution @IzhakJakov and for the review @andytom! 😄
Thank you :) |
Description
Modify
status
module work even when the status is 0Motivation and Context
Closes #3746 and enables a potential workaround for #2967 as well.
Screenshots (if appropriate):
How Has This Been Tested?
Checklist: