Generic export functionallity

RESOLVED FIXED

Status

Firefox OS
Gaia::Contacts
P1
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: arcturus, Assigned: arcturus)

Tracking

({feature})

unspecified
x86
Mac OS X
feature
Dependency tree / graph
Bug Flags:
in-moztrap +

Firefox Tracking Flags

(blocking-b2g:koi+)

Details

(Whiteboard: [u=commsapps-user c=contacts p=5])

Attachments

(1 attachment)

We have new US for exporting contact to the SIM card and to vcard.

We should try to do as well as we did for the import contacts, provide a common mechanism to 'export' to 'any' source, and then enable 'connectors' (as could be the vcard or the sim ones to use that common UI.
Blocks: 887673
No longer depends on: 887673, 888931
Blocks: 887776
listed as must have in v1.2 for COMM team. koi+
blocking-b2g: koi? → koi+
Hi,

as a generic export functionality, we are thinking of a common ui that allow the user to export a single, or multiple contacts in the different formats.

It comes to my mind like the contacts list we use from importing from different sources, perhaps we could even reuse that one.

Ayman, Jose Manuel, what do you think about this?
Flags: needinfo?(jmcf)
Flags: needinfo?(aymanmaat)

Comment 3

4 years ago
yes, I'm sympathetic with that idea, and in this case the UI can be more simple than with import as the user will only have to select and not to unselect. So in  the end we can have an architecture based on exporter_ui, exporter connectors and exporters.
Flags: needinfo?(jmcf)

Comment 4

4 years ago
In case we have a new UI for the US, we need some test cases for it. Minus in-testsuite if we decide not to have a new UI for it.

Updated

4 years ago
Flags: in-testsuite?

Updated

4 years ago
Flags: in-testsuite? → in-moztrap?

Comment 5

4 years ago
MozTrap #9047, #9048
Flags: in-moztrap? → in-moztrap+
Whiteboard: [ucid:Comms5], [u=commsapps-user c=contacts p=0] → [ucid:Comms5 KOI:P1], [u=commsapps-user c=contacts p=0]
Should this really be an user story?
The user stories seems to be bug 887673 and bug 887776 saying the export of contact to USIM and Media Card

This bug sounds like the underlying generic solution to support both user stories so this bug can probably be used to land the actual foundation for both bug 887673 and bug 887776. 

What do you think? thanks
Flags: needinfo?(noef)
@joe,

we want to put work here as an user story to deliver a functionality that will help us not just to deliver bug 887673 and bug 887776, but to allow us to include in the future any kind of source to export to (like could be any external service: google contacts).

That's the reason we want to invest some time here, to save it later ;)

Thanks!
yes i totally understand. 
i was cleaning up the user story bugs and try to make a better user story tree in bugzilla.
You are definitely welcome to take a generic approach to satisfy the 2 user story bugs. i was just feeling that this is not exactly an user story (in a format that's put together by the Product team). so i think this should only be linked to bug 887673 and bug 887776, rather than linking to the whole v1.2 COMMs meta bug 887455.
it's just a minor dependency tree linking thing :)
sure :)

Thanks Joe!
No longer blocks: 887455
Whiteboard: [ucid:Comms5 KOI:P1], [u=commsapps-user c=contacts p=0] → [u=commsapps-user c=contacts p=0]

Updated

4 years ago
Flags: needinfo?(noef)
Summary: [User Story] Generic export functionallity → Generic export functionallity
Assignee: nobody → francisco.jordano
Whiteboard: [u=commsapps-user c=contacts p=0] → [u=commsapps-user c=contacts p=5]
Depends on: 901079
Depends on: 902873
No longer depends on: 902873

Updated

4 years ago
QA Contact: atsai
Created attachment 790797 [details]
Pointer to PR 11550

This is the generic functionality, enabling the panels in settings, navigation, and contacts selection.

@Ben, I'll submit a branch with this code an an example of an strategy working, the SIM export, which will be used as PR for bug 887673

Thanks!
Attachment #790797 - Flags: review?(bkelly)
@Ben, I've uploaded the code for testing this genereric exporter to this branch:

https://github.com/arcturus/gaia/tree/bug-887673

That is using this code + the SIM export strategy which will be part of PR for bug 887673 once added unit tests.

Thanks!
F.

Comment 12

4 years ago
Initial comments in the Github PR.  I still want to run the tests and try it out on my phone.  I won't have time to do that until the morning, though.
Status: NEW → ASSIGNED

Comment 13

4 years ago
Comment on attachment 790797 [details]
Pointer to PR 11550

Francisco, I think this looks good.  I like how the export code is coming together.  Since there are just a few test issue to address I'm going to go ahead and r+.  We just need to make sure the tests pass on travis before landing.

Nice work and thanks!
Attachment #790797 - Flags: review?(bkelly) → review+

Comment 14

4 years ago
removing ni to me as comment 2 was resolved in discussion a while ago
Flags: needinfo?(aymanmaat)
Landed:

https://github.com/mozilla-b2g/gaia/commit/e19945e75290c9c8731456af5cdc526a4a492ad1
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.