Skip to content

Conversation

@lewiscollard
Copy link
Contributor

@lewiscollard lewiscollard commented Nov 14, 2019

This was previously done in #175. I prefer this implementation because:

  1. It fixes it in one place, not two
  2. It's addressing the problem, which is Watson throwing an exception on .content, in a narrower scope
  3. It doesn't quietly swallow ObjectDoesNotExist, which is something that we absolutely do want to be thrown if it happens.

@lewiscollard lewiscollard changed the title Fix ObjectDoesNotExist exception recovering pages with Reversion: reloaded Fix ObjectDoesNotExist exception recovering pages with Reversion (reloaded) Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants