Closed 3ygun closed 7 years ago
@tylermzeller let me know if you think this is a "good" direction and from there I'll complete the remaining issues.
This is long overdue. Good work. I'm going to merge it when I get home. My only remaining concern is why there was a duplicate server file in the first place. I had just assumed it needed to be different from the iOS server file, but that's something we will need to ask Jihun.
Goals
Cleanup the server enhancing process understanding and option editing. Enabled through refractoring into functions and a
configuration.json
file.Parts
crowdML-server-Android.js
filecrowdML-server.js
to:configuration.json
discussed belowConstants.js
intoconfig/configuration.json
crowdML-server.js
which can be solved by having master config object which is initialized to the values ofconfiguration.json
and modified from there.Remaining Problems
\/ Extracted to #20 \/