Merge lp://staging/~jonas-drange/ubuntu-system-settings/apn-prototype into lp://staging/ubuntu-system-settings

Proposed by Jonas G. Drange
Status: Merged
Merged at revision: 1488
Proposed branch: lp://staging/~jonas-drange/ubuntu-system-settings/apn-prototype
Merge into: lp://staging/ubuntu-system-settings
Diff against target: 3490 lines (+2067/-1143)
16 files modified
plugins/cellular/CMakeLists.txt (+3/-4)
plugins/cellular/Components/CMakeLists.txt (+3/-0)
plugins/cellular/Components/KeyboardRectangle.qml (+74/-0)
plugins/cellular/Components/LabelTextField.qml (+54/-0)
plugins/cellular/Components/StandardAnimation.qml (+22/-0)
plugins/cellular/PageApnEditor.qml (+394/-346)
plugins/cellular/PageChooseApn.qml (+333/-535)
plugins/cellular/apn_editor.js (+192/-0)
plugins/cellular/apn_manager.js (+588/-60)
plugins/cellular/ofonoactivator.cpp (+0/-132)
plugins/cellular/ofonoactivator.h (+0/-42)
plugins/cellular/plugin.cpp (+0/-2)
tests/autopilot/ubuntu_system_settings/__init__.py (+147/-1)
tests/autopilot/ubuntu_system_settings/tests/__init__.py (+17/-21)
tests/autopilot/ubuntu_system_settings/tests/ofono.py (+99/-0)
tests/autopilot/ubuntu_system_settings/tests/test_cellular.py (+141/-0)
To merge this branch: bzr merge lp://staging/~jonas-drange/ubuntu-system-settings/apn-prototype
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot continuous-integration Needs Fixing
Alfonso Sanchez-Beato mms/nuntium interactions Pending
Tony Espy sanity Pending
Matthew Paul Thomas design Pending
Review via email: mp+258992@code.staging.launchpad.net

Commit message

[cellular/apn] new APN panel made for the new 'Preferred' property, as well as the 'ResetContext' method.

Description of the change

New UI due to new NetworkManager, Nuntium and Ofono backend.

Silo: 33

* Is your branch in sync with latest trunk (e.g. bzr pull lp:trunk -> no changes)
Yes
 * Did you build your software in a clean sbuild/pbuilder chroot or ppa?
Yes
 * Did you build your software in a clean sbuild/pbuilder armhf chroot or ppa?
Yes
 * Has your component "TestPlan” been executed successfully on emulator, N4?
No, it's currently being written AND the NetworkManager backend change has not landed yet.
 * Has a 5 minute exploratory testing run been executed on N4?
N/A
 * If you changed the packaging (debian), did you subscribe a core-dev to this MP?
Yes, kenvandine
 * If you changed the UI, did you subscribe the design-reviewers to this MP?
mpt
 * What components might get impacted by your changes?
NetworkManager, Nuntium, Ofono
 * Have you requested review by the teams of these owning components?
Yes

To post a comment you must log in.
1409. By Jonas G. Drange

revert changelog

1410. By Jonas G. Drange

no auto in username, next field to be focused after pwd is name

1411. By Jonas G. Drange

merge trunk

1412. By Jonas G. Drange

fix changelog

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1413. By Jonas G. Drange

ap proxy for labeltextfield, as well as some minor jiggling of the test

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
1414. By Jonas G. Drange

merge trunk

1415. By Jonas G. Drange

make contexts appear as active

1416. By Jonas G. Drange

refactor the test class, divide tests and create some new ones

1417. By Jonas G. Drange

some margin

1418. By Jonas G. Drange

fixing some review killers

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1419. By Jonas G. Drange

fixing bad modem path

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
1420. By Jonas G. Drange

add ready signal and properly deal with active contexts when there are no preferred ones

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
1421. By Jonas G. Drange

remove ofonoactivator and sessions service deps

1422. By Jonas G. Drange

fix reset function

1423. By Jonas G. Drange

make the ui support changing type of context

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
1424. By Jonas G. Drange

address comments from design

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1425. By Jonas G. Drange

dropped Base in favor of Standard

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
1426. By Jonas G. Drange

move reset

1427. By Jonas G. Drange

logic for conflicting contexts, as well as the feedback to the user should it make a mess

1428. By Jonas G. Drange

note to translators about APN field label

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Just a few inline comments on strings that we probably don't need translated, otherwise looks good.

review: Needs Information
Revision history for this message
Jonas G. Drange (jonas-drange) wrote :

Comments about translations. Thanks!

1429. By Jonas G. Drange

warning on de-prefer

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1430. By Jonas G. Drange

merge trunk

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1431. By Jonas G. Drange

removing debugging

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1432. By Jonas G. Drange

sync with trunk since this branch is old

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

I'm happy with this branch, wish we could see passing tests in CI but right now we are affected by an autopilot bug.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches