View source for Gaia/New-Architecture-Program/Contacts/FxOS-S7(18 Sep)

== General info ==
*Members
**Borja Salguero (DEV)
**Fernando Campo (DEV)
**Manuel Casas (DEV)
**Maria Oteo (EPM)

*Meetings:
**Daily happening everyday in Mumble at 10:30 CEST
**Sprint Planning, on Monday (every two weeks)
*Links of interest:

== Sprint objectives ==
*Separation in views is also at risk for 2.5 as it seems that the pre-rendering proposal made by Vivien ([https://bugzilla.mozilla.org/show_bug.cgi?id=1190805 Bug 1190805]) is not going to be landed in System. Platform team have another ideas about how implementing as part of the Pre-rendering ([https://bugzilla.mozilla.org/show_bug.cgi?id=730101 Bug 730101]) it but they don't have enough time to do it in 2.5 as they are focused on landing SWs in Desktop and Android. Several proposals are being handled but no decision has been made yet.
**For the time being, we have been asked to NOT continue landing view separation for settings, list and the overlay.It seems that we have to be ready to not to have the navigation in separated views (possible rollback of #open and #update views in case the pre-rendering is not in time for 2.5 or the white flash while navigating can not be fixed)
**Content wrapper has also been suggested as a navigation alternative but it won't be the panacea, gecko has problems loading several documents at the same time. We can perform tests to avoid to do everything at the same time (just like prerendering does), but that's not related to sw or prerendering. Besides it implies more work, view separation + content wrapper and there are doubts about the memory consumption:
***No prerendering support in the platform and mocking it at the app level could imply memory problems. With platform support, we don't prerender anything if we have not enough memory. But we can't emulate that on the app side as we have no idea of the current memory consumption. So we will always be prerendering the next screen no matter what memory we have.
**Gecko team requested a profiling of the navigation problem (e.g. white flash when moving from a document to another document) so they can be aware of them and try to find an easier fix. Borja and Julien will work on it.
***Julien has created the bug and has added the profiling info [https://bugzilla.mozilla.org/show_bug.cgi?id=1199674 Bug 1199674] , Platform team is looking at it to see if to see why we're slow at changing pages and if a patch could replace the use of the pre-rendering.
**When Francisco comes back next week from his holidays and [https://bugzilla.mozilla.org/show_bug.cgi?id=1199674 Bug 1199674] is investigated, it will be decided what to do.

=== Bugs for this sprint ===
*Pending bugs to be fixed in FxOS-S7 (18 Sep) target milestone:https://goo.gl/RkgKKR
**Pending bugs to be fixed in FxOS-S7 (18 Sep) target milestone that already have a patch:  https://goo.gl/LQ8AZj
<bugzilla>
    {
        "product":"Firefox OS",
        "target_milestone":"FxOS-S7 (18Sep)",
        "component":["Gaia::Contacts"],
        "whiteboard":"NG Gaia Contacts",
        "whiteboard_type":"contains",
        "include_fields": "id, summary, status, resolution, assigned_to, depends_on, blocks, whiteboard, cf_feature_b2g, cf_blocking_b2g, target_milestone"
    }
</bugzilla>

== Regressions introduced by our work (highest priority) ==
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1191471 Bug 1191471] - [Contacts] Creating a new contact has no default fields populated. '''Francisco to confirm if it's a bug or not'''
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1191606 Bug 1191606] - [Contacts][Import] Contact thumbnails are improperly cropped to the upper left portion of the image on the import screen ('''FerJ''') '''2.5+'''
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1192180 Bug 1192180] - [Contacts]It can't enter contact details page. ''' Feature 2.5+''' Regression or better, extra feature. '''NI to Francisco'''
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1183645 Bug 1183645] - Performance regression in Contacts
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1201419 Bug 1201419] - [NFC] Unable to share contact via NFC ('''FerJ''') '''2.5+'''
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1201052 Bug 1201052] - After updating a contact the statusbar turns from blue to grey ('''Borja''')
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1202217 Bug 1202217] - [Contacts]Merge contact B to contact A, but the contact B is also displayed. ('''Borja''') '''2.5+'''

== Other bugs to pay attention ==
*List of 2.5+ bugs in Contacts: https://goo.gl/En93Ek

=== other non-blocking bugs related to import functionality ===
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1152237 Bug 1152237] - [Flame][Message]Device will enter contacts list page when you tap the .vcf (without name and last name) to view. '''Invalid''', the issue will be tracked in [https://bugzilla.mozilla.org/show_bug.cgi?id=1192180 Bug 1192180].
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1124095 Bug 1124095] - Incorrect message when receiving contacts via Bluetooth
== Bugs we need to revisit when a decision is made about the view separation in Contacts  ==
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1191124 Bug 1191124] Refactor Contacts to use modern L10n API
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1156192 Bug 1156192] - [NFC] Dialer show shrinking UI when contact (shareable object) running on the background
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1190543 Bug 1190543] - [Contacts] Opening Settings shows all buttons as enabled briefly before then disabling all relevant buttons
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1196893 Bug 1196893] - [Contacts] Navigating to Contact Details flashes blank communications-colored page for too long

== Issues found when landing our work but other FxOS teams have to fix ==
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1188350 Bug 1188350] - When opening a URL with window.open, the title is not set properly. '''Set as 2.5+'''
**Issue found by Borja in system frontend that can be reproduced in master and specially when applying the patch in [https://bugzilla.mozilla.org/show_bug.cgi?id=1182509 Bug 1182509] - [Contacts][NGA] Open contacts_matching with window.open
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1184953 Bug 1184953] - [Web Components] Document's element registry is not persisted in the BF Cache
**Bug ssue detected when working in [https://bugzilla.mozilla.org/show_bug.cgi?id=1183727 Bug 1183727] - [Contacts][NGA] Create #update view and connect it to #open within Contacts App
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1191288 Bug 1191288] - Broken UI in Contacts matching screen
**It's due to [https://bugzilla.mozilla.org/show_bug.cgi?id=1168951 Bug 1168951] - [User Story] Pin Page, so System FE team should take care of it.
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1198522 Bug 1198522] - Long pressing on an image in edit mode will bring up a menu that does not function.

*[https://bugzilla.mozilla.org/show_bug.cgi?id=1202359 Bug 1202359][Contacts]Tap the check box(small rectangular box) in "Duplicates Found" view for once, actually it performed double taps.
**It seems similar to the already resolved by Etienne [https://bugzilla.mozilla.org/show_bug.cgi?id=1190775 Bug 1190775] - [System] If an activity perform a window.open, the popup will be unresponsive. NI to Etienne for more info.
**Confirm if it has raised after landing [https://bugzilla.mozilla.org/show_bug.cgi?id=1182509 Bug 1182509] - [Contacts][NGA] Open contacts_matching with window.open. ni to Etienne if no answer we should ask for the regression window.

== Daily meetings ==
*Tracking daily bugs in this [https://etherpad.mozilla.org/bugsupdatecontacts Etherpad]
== Issues during the sprint ==
=== Meeting to analyze the different options for the NGA work (9/9/2015) ===
*After some investigation in Bug 1199674, it seems that there are not too many options to fix that issue in 2.5 without Pre-rendering.
*For that reason, in 2.5 Contacts appplication (at least the official one), we will not land more separated views (so we will not land settings and list views in master).
*What options do we have now?
**If Product and UX approve it, we will leave master as it's right (with view and update views separated). For that, they need to analyze if the user experience is good enough with aries and flame device (worse case).
**If they consider that it's necessary the roll back, we should analyze the commits from https://github.com/mozilla-b2g/gaia/commit/a1916008434c874f2705fe854d41234fa14db63c#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6 that corresponds to Bug 1183727 (Update view) and decide what commits shoud be backed out. Apart of that, we need to decide what we do with our work of separating settings and list views (that it's not landed in master but we have already done in a demo branch)
***Include it in dev-app. 
***Include it in another different branch
**In both options, it's necessary to update the branch with the fixes that will go landing in master otherwise it will be very difficult that the already work can be reused
*Francisco has arranged a meeting today (9/9) with UX and Product team to see what they decide.
*It's important to stress that if the roll back is decided as soon as we done it the better.
*In the meantime the NGA Contacts team will analyze the commits to decide what should be the candidates for the back out and also we will update the demo branch with master + settings and list separated views.

=== Analyzing the candidates commits for the possible backout ===
*https://github.com/mozilla-b2g/gaia/commit/6da9b222cb0dce9a7b7bfafff7b41e412ae863cb#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6
**Bug 1201419 - [NFC] Unable to share contact via NFC
*https://github.com/mozilla-b2g/gaia/commit/b8f5afcbbdaed9edc15550b31db0186d57736b7f#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6
**Bug 1201052 - After updating a contact the statusbar turns from blue to grey
*https://github.com/mozilla-b2g/gaia/commit/1e206e7db8c2f2c7f9c05f239936700af6329cff#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6
**Bug 1202217 - [Contacts][NGA][Detail] Merge contact B to contact A, but the contact B is also displayed.
*https://github.com/mozilla-b2g/gaia/commit/1e206e7db8c2f2c7f9c05f239936700af6329cff#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6
*https://github.com/mozilla-b2g/gaia/commit/51a8344093e0391748530b1fe182a77dad423beb#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6
*https://github.com/mozilla-b2g/gaia/commit/a5451e854daf66c0b066a0761d53fd1a94b8744a#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6
*https://github.com/mozilla-b2g/gaia/commit/a1916008434c874f2705fe854d41234fa14db63c#diff-2e7b8b1dd0ec1da64a77f6a6e1330aa6

== Demos ==

== Retrospective ==
=== Actions taken from last sprint ===
=== Things that went well ===
=== Things that went not that well ===
=== Actions for this sprint (apart of the Sprint Objectives) ===