approve sub request changes the shift to the other user; closes #910

breaking this out into it's own method makes for more lines of code.  May as well inline it