Remove call to SetUpgradeErrorReceipt from restoreChannel.

[chatton]

Remove call to SetUpgradeErrorReceipt from restoreChannel.

This feels like a side effect, and might be cleaner to call separately afterwards.

Originally posted by @damiannolan in #5237 (comment)

[vishal-kanna]

@chatton Can i work on this issue?

[chatton]

sure @vishal-kanna, this one should be going into the 04-channel-upgrades feature branch, thanks!

[vishal-kanna]

@chatton Is it removing the SetUpgradeErrorReceipt from restoreChannel .
Do we need to add it anywhere else?

[chatton]

@vishal-kanna yeah we'll need to add it everywhere that restore channel is currently being called to ensure the behaviour is the same. It looks like that is

1. here
2. here
3. And here

There may be some tests that fail after you move these things, I can help out with the refactors of the tests if you run into this issue!