-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
API V3: clean version when deactivated and build version when activated #10308
Conversation
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.
Nicely done!
readthedocs/api/v3/views.py
Outdated
if not was_active and version.active: | ||
trigger_build(project=version.project, version=version) | ||
# Purge the cache from the CDN. | ||
version_changed.send(sender=self.__class__, version=version) |
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.
That seems like an additional bug fix in the mix 馃憤
Co-authored-by: Benjamin Balder Bach <benjamin@readthedocs.org>
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.
I didn't review it deeply because it was approved already. However, I checked the "shared logic" which was what I asked for and it's 馃憤馃徏 . Thanks.
This matches what we do in our version form.
Closes #10221.
馃摎 Documentation previews 馃摎
docs
): https://docs--10308.org.readthedocs.build/en/10308/dev
): https://dev--10308.org.readthedocs.build/en/10308/