Code review comment for lp:~allanlesage/address-book-app/abook_navigation_favorites

Revision history for this message
Chris Gagnon (chris.gagnon) wrote :

holding back on this MP until a bug is fixed is blocking other apps from landing tests. like the dialer and the messaging-app. When the in-memory backend is fixed we can change the helper to use it instead of using the address-book-service.

We want one place to add and remove contacts, so we don't have to update every project when there is a change. the address-book-app is the most logical place to keep the helper.

The tests are working currently, there is no reason to block landing the tests from landing.

« Back to merge proposal