@esheffield  not sure what you mean. It shouldn't matter if we end up overwriting the values in the dict.

@isethi This is how it was when i moved it out of the try-except block and it's done like this everywhere else. The only real difference is that it's a copy but not sure that matters. I think if we were gonna change it we should have the deserialize_schedule_metadata return values instead of changing the body in place but that's arguably a separate patch since it doesn't affect the goals of this patch.