it looks like these two lines would raise an exception if indicator_updated was a "falsey" non-None object, like an empty string. it may be that that's never the case, but we should still probably handle it more explicitly