Remove local setting Load Production Certificate about pacific-emis-education-survey-tool HOT 11 OPEN

@maratomega Please share progress on this one.

from pacific-emis-education-survey-tool.

@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.

from pacific-emis-education-survey-tool.

@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?

from pacific-emis-education-survey-tool.

In meeting it was discussed that:

• We may be able to remove this local settings without crashing the app with the proposed approach in #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)

from pacific-emis-education-survey-tool.

@maratomega what is the progress on this?

from pacific-emis-education-survey-tool.

Completed

from pacific-emis-education-survey-tool.

@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?

from pacific-emis-education-survey-tool.

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

from pacific-emis-education-survey-tool.

@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?

from pacific-emis-education-survey-tool.

@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.

from pacific-emis-education-survey-tool.

@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?

from pacific-emis-education-survey-tool.