Fix closing account confirmation modal

[YanaMIvanova]

Q	A
Bug fix?	Yes
New feature?	No
BC breaks?	No
Fixed issues	#1964

This PR fixes issue #1964

[Elorfin]

You can't just comment code and hope everything is still working.
IMO, the real problem is explained in comment above the commented code : it will throw errors if the state is mistakenly mutated.

FI, I don't reproduce your problem on my platform.

[maieutiquer]

It's reproduced when NODE_ENV is "development", which is only set in our environment:

But this explains why we didn't get this error before - we recently added NODE_ENV to our env variables

If you cannot reproduce, this might also mean that it never worked for you in development mode neither

I believe mutations should print a warning, but not crash the application, or at least it should be fixed before blocking further development of features

[Elorfin]

Whatever you set as NODE_ENV, it is overidden by webpack. https://webpack.js.org/configuration/mode/
The problem only occurs when the registration mail option is set to another value than "none".

[maieutiquer]

Whatever you set as NODE_ENV, it is overidden by webpack. https://webpack.js.org/configuration/mode/ The problem only occurs when the registration mail option is set to another value than "none".

This seems to be the case, so what would be the solution for this?