Moved the code inside finally block out, so is_rr_approved is referenced after the whole try/except block.

I did some manual testing on reviews requests that are shipped and not shipped. 
The code passes all existing unit tests.