Remove local setting Load Production Certificate

[ghachey]

This setting is no longer required as both the production and development service account keys are loaded from the centralized configuration. Verify it is in fact no longer used and remove.

[ghachey]

@maratomega Please share progress on this one.

[maratomega]

@ghachey if we will remove this certificate, the app will crash if the user won't have the internet connection. Tested on our side without internet, it crashes once the user selects the country.
We can keep this cert without any harm, but if you require us to remove it we will need to show some error pop-up

Please share your thoughts.

[ghachey]

@maratomega ok we can keep. But it looks like an old key. Can you try replacing the key with the one I sent you through Skype.

I believe the place to replace is app/src/main/assets/c_dev.json

And see if it still all behaves correctly. Also, this could be related to issue #68 as this old server_account key is from the "Education Survey Tool Base" project. Is this key used in any way once the country is selected and a production key retrieved?

[ghachey]

In meeting it was discussed that:

• We may be able to remove this local settings without crashing the app with the proposed approach in Support to add countries through configuration #68 (comment)
• If so, please also let's try to remove the hard coded keys in the source code (e.g. app/src/main/assets/c_dev.json)

[ghachey]

@maratomega what is the progress on this?

[maratomega]

Completed

[ghachey]

@maratomega I installed the latest build and I still see in the settings "Load PRoduction Certificate", is there a reason why this local settings is still there? I thought you said this issue is complete, can you elaborate a little?

[maratomega]

@ghachey we haven't removed from the settings "Load Production Certificate" because of issues with the internet connection, it prevents the app from crashing.

[ghachey]

@maratomega I thought you said following the approach of per country builds would no longer require this old key to keep the app from crashing without Internet?

[maratomega]

@maratomega I thought you said following the approach of per country builds would no longer require this old key to keep the app from crashing without Internet?

During the testing without it, on the per country builds, we still had that issue.

[ghachey]

@maratomega I thought you said following the approach of per country builds would no longer require this old key to keep the app from crashing without Internet?

During the testing without it, on the per country builds, we still had that issue.

Can you at least hide the config from the user local settings?