-
Notifications
You must be signed in to change notification settings - Fork 26
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
Forcing users to restart if there is any update detected. #407
Conversation
This applies to both manual as well as addon-updater updates. The reason for this change is to avoid errors that will happen in future versions when we rename modules, remove functions, and generally revamp the structure of MCprep. Many users encounter errors due to not restarting Blender already, even with 'typical' updates.
@@ -845,6 +903,10 @@ class MCPREP_PT_bridge(bpy.types.Panel): | |||
bl_category = "MCprep" | |||
|
|||
def draw(self, context): | |||
if addon_just_updated(): |
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.
It would be nice if there was a short comment above these explaining what they do for any future developers
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.
Do you think it's sufficient to have that comment in the code of those functions? Otherwise we'll be having copies of the comment in 10x places, adding to the boilerplate.
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.
Should be fine as long as it's explained what the functions do and why they do that they do
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.
So... would you modify the docstrings that are there now I guess is the question :)
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.
Just a couple of minor things needed
Looks good now |
This applies to both manual as well as addon-updater updates. The reason for this change is to avoid errors that will happen in future versions when we rename modules, remove functions, and generally revamp the structure of MCprep. Many users encounter errors due to not restarting Blender already, even with 'typical' updates.
closes #402