<div dir="ltr"><div><div>Hello,<br><br>I've just send a pull request with the implementation of 3G database support in control panel modem configuration section.<br><br><a href="https://github.com/sugarlabs/sugar/pull/46">https://github.com/sugarlabs/sugar/pull/46</a><br>

<br>In this email, I'm attaching  a screenshot to review UI design.<br><br></div><div>About the implementation, some highlights:<br><br>- There is a new config.py file with constants: file paths and GConf keys.<br><br>

- It uses GConf keys to persistently store country, provider and plan selections<br><br></div><div>- In model.py a class controller retrieve data form XML and store selections on GConf<br><br></div><div>- In the view.py, adds a new ScrolledWindow with an upper box for 3 combo provider selection, a separator and a lower box with the previous entries for network settings.<br>

</div><div><br></div><div>- When a plan is selected, network settings are changed using the same methods.<br></div></div><div><br></div><div><br>I have some concerns:<br><br>- Is the GUI correct?<br><br>- File paths should be move to a template <a href="http://config.py.in">config.py.in</a> file?<br>

</div><div><br></div><div>- I have kind of unitary tests for ServiceProviderDatabase class but I don't know exactly where to put them. They use mock library to simulate GConf calls.<br><br></div><div>- Is it necessary to write functional test for view? Is there information about how to do it?<br>

</div><div><div><br><br clear="all"><div>Thank you for your attention.<br></div><div>-- <br><div dir="ltr">Miguel González<br>Activity Central: <a href="http://www.activitycentral.com" target="_blank">http://www.activitycentral.com</a><br>

</div>
</div></div></div></div>