[User Story] Send contact as a VCARD via bluetooth

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: wmathanaraj, Assigned: fcampo)

Tracking

({feature})

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

Firefox Tracking Flags

(blocking-b2g:koi+, b2g-v1.2 fixed)

Details

(Whiteboard: [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=4][Sprint 4])

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
User Story:

As a user I want to select a contact from the contact list and send to another device that is already paired via Bluetooth with the sending device.
As the sender if bluetooth is not switched on I want to be informed and given the option to switch on bluetooth on the device. After switching on bluetooh, I want to automatically return to the contact sending process.
As the sender, if I am not already paired with the receiving device, I want to be given the opportunity to pair. After pairing, I want to automatically return to the contact sending process.

Preconditions:

* Contact app is open
* Sending and receiving devices have BT capability 

Acceptance Criteria:

* user is informed of successful sending of contact
* any sending failure is presented to the user in a comprehensible message
On the sender side's Contact app

 - pair phone with receiver device, if necessary;
 - create a file (extension is .vcf) with the VCARD data;
 - send the VCARD file to the paired receiver; and
 - display success or error message.

There should already be example code for some of these steps in the Music app.

See bug 887663 for the receiver side.
Adding also Sergi Mansilla, as he has been working on the current import from vcard.
A Pivotal Tracker story has been created for this Bug: http://www.pivotaltracker.com/story/show/52538267
David Scravaglieri deleted the linked story in Pivotal Tracker
(Reporter)

Updated

5 years ago
Depends on: 889304
We have already had a centralized app in charge of Bluetooth file transferring. Any application which wants to send a file via Bluetooth can simply communicate with that app via mozActivity named 'share'. Please see https://github.com/mozilla-b2g/gaia/blob/master/apps/gallery/js/gallery.js#L1066 for more information.
Whiteboard: [ucid:Comms3] → [ucid:Comms3], [u=commsapps-user c=contacts p=0]
listed as must have in v1.2 for COMM team. koi+
blocking-b2g: --- → koi+

Updated

5 years ago
Flags: in-moztrap?

Comment 7

5 years ago
MozTrap #9045
Flags: in-moztrap? → in-moztrap+
There is a second way to transfer business cards between phones.

The Bluetooth Object Push Profile has built-in support for transferring vCards. We'd need to extend our support for OPP to use this facility, but that shouldn't be much work. See [1], Section 3 and 4. This will probably be compatible with a lot of phones, so I more and more think that using OPP is the better approach.

[1] https://www.bluetooth.org/docman/handlers/downloaddoc.ashx?doc_id=230913
But then again, this spec reads like it'd only support transferring a 'default business card' of the phone's owner.

Eric, could you have a look at the OPP spec and give your opinion? Do you think it is generic enough for our purposes?
Flags: needinfo?(echou)
(In reply to Thomas Zimmermann [:tzimmermann] [:tdz] from comment #8)
> There is a second way to transfer business cards between phones.
> 
> The Bluetooth Object Push Profile has built-in support for transferring
> vCards. We'd need to extend our support for OPP to use this facility, but
> that shouldn't be much work. See [1], Section 3 and 4. This will probably be
> compatible with a lot of phones, so I more and more think that using OPP is
> the better approach.
> 
> [1] https://www.bluetooth.org/docman/handlers/downloaddoc.ashx?doc_id=230913

I think we should open a different bug for that feature you comment Thomas. Which will require setting up a default identify for the phone, something that we don't have so far.

Also from doing this we were thinking about web activities as well, sending a single vcard file containing the desired contacts to export.

Cheers,
F.
(Reporter)

Comment 11

5 years ago
This spec is very much focused on owner's business card rather than sending any contact as vcard to another device.
Hi Thomas,

There are 3 features defined in OPP spec:

(1) Object Push
(2) Business Card Pull
(3) Business Card Exchange Feature

In this bug, we are asked to support "sending(push) a contact from the contact list to the remote device via Bluetooth". To achieve this, all we need is (1).

We can see the vCard file exported from Contacts app as a common 'object'. Apparently feature (2) is not we are looking for. As for feature (3), it's like two people exchanging their business cards. However, it's not what we want. What we want is to let the user be able to select an arbitrary contact from Contact app and send to a paired Bluetooth device, just like Wilfred mentioned in comment 11. Moreover, feature (1) can be also used on sharing the owner's business card to any paired Bluetooth device. That's why I said that Gecko Bluetooth is ready for this user story.
Flags: needinfo?(echou)
Whiteboard: [ucid:Comms3], [u=commsapps-user c=contacts p=0] → [ucid:Comms3 KOI:P1], [u=commsapps-user c=contacts p=0]
Assignee: nobody → noef
Whiteboard: [ucid:Comms3 KOI:P1], [u=commsapps-user c=contacts p=0] → [UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0]

Updated

5 years ago
Assignee: noef → nobody
Whiteboard: [UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0] → [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0]

Updated

5 years ago
Whiteboard: [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0] → [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0][Sprint 2]

Comment 13

5 years ago
Created attachment 784939 [details]
FFOS_ContactsExport_20130801_V1.0.pdf

Wireframe release:

__RELEASE NOTE__

new wireframes
- none

updated wireframes
- none

deleted wireframes
- none

new flows
- Send via Bluetooth : Single VCard
- Send via Bluetooth : Multiple VCards
- Receive via Bluetooth : VCard

updated flows
- none

deleted flows
- none

Updated

5 years ago
Depends on: 901471
Hi Ayman,

I have a suggestion about the flow. When the user has done choosing the contact he wants to share and is going to select a Bluetooth device, it doesn't always need to search nearby devices and go through the pairing process. Just like Settings app, there should be another area called 'Paired Devices' for users to select paired devices. 

Hope this will help.

Eric

Comment 15

5 years ago
Created attachment 785752 [details]
FFOS_ContactsV1.2_20130805_V2.0.pdf

pages 5 to 7 of this wireframe pack are relevant to this bug

Wireframe pack V2.0 release note:

updated flows

Send via Bluetooth : Single VCard
- step 3 removed
- FB statement added to step 2
- SIM and Vcard CTAs added to step 2

Send via Bluetooth : Multiple VCards
- step 4 removed
- FB statement added to step 3
- spinner added between selection of export cta and export process beginning
Attachment #784939 - Attachment is obsolete: true
Please find all visuals required for this implementation in the following link:
For Single VCard: https://www.dropbox.com/sh/bkvv86ad8ed5ugi/kJTR61A1fO

for Multiple VCard: https://www.dropbox.com/sh/1yv2pqfnaugiioc/UeMT8eHrA3

If you have any question, please do not hesitate.
Victoria

Comment 17

5 years ago
Working on defining the test cases
Assignee: nobody → isabelrios

Updated

5 years ago
Whiteboard: [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0][Sprint 2] → [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0][Sprint 3]
(Assignee)

Updated

5 years ago
Assignee: isabelrios → fernando.campo
(Assignee)

Updated

5 years ago
Depends on: 896982
(Assignee)

Comment 18

5 years ago
Updating the estimated time for the bug. As it implies new UX and processes, I don't expect to finish it before middle next week.
Whiteboard: [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=0][Sprint 3] → [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=4][Sprint 3]
Per comment 18 moving to Sprint 4.
Whiteboard: [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=4][Sprint 3] → [TEF][UCID:Comms3, FT:comms, KOI:P1], [u=commsapps-user c=contacts p=4][Sprint 4]
We've seen that we will need to create a temporary file on the sdcard to send that file through bluetooth.

There is a bug already for avoiding this:

https://bugzilla.mozilla.org/show_bug.cgi?id=821186

Meanwhile we will work around the problem creating that temporary file until this is solved and we can send directly the blob.

Thanks,
F.
Ayman/Carrie, 

Regarding contacts over Bluetooh, contacts should be sent via the Bluetooth file transfer app and therefore the flow will be the same as Bug 892212 - [B2G] [BT USer Story]: Support for BT inline pairing User

Regarding the inline paring UX, base on BT team's discussion, in order to close this story before v1.2 feature complete date, Carrie's original proposal is considered as the most feasible one for eng. "auto-pairing" in Ayman's proposal is not likely to make it before v1.2 feature closes.

For v1.2, let's proceed with providing a comparatively better UX by adding a few missing pieces base on the current flows (Carrie's original proposal) while you can continue discussing future improvements on this.

Hopefully this sound like a reasonable path forward. thanks
(Assignee)

Comment 22

5 years ago
(In reply to Joe Cheng [:jcheng] from comment #21)
> Ayman/Carrie, 
> 
> Regarding contacts over Bluetooh, contacts should be sent via the Bluetooth
> file transfer app and therefore the flow will be the same as Bug 892212 -
> [B2G] [BT USer Story]: Support for BT inline pairing User
> 
After checking the bug out, I reckon that the flow would work properly when we want to export one contact
Contact list > Choose a contact > On contact details, choose export > check BT > pair > send.

But AFAIK, this won't work when exporting from Settings (possibility of exporting multiple contacts), where the flow is:
Contact Settings > export > bluetooth > (should check if bt is on, and give option to turn on if not) > pair with device > choose contacts from the list > send
...as the current activity 'share' for bluetooth requires the data before checking if the bluetooth is connected.

One solution for this would be create another activity for bluetooth where we just check the availability of the bluetooth, and if is off we can let the user turn it on. That would help with the flow for multiple contact export that Ayman proposes.

I'd like to hear your opinions, and if agree, maybe Joe can put me in contact with BT team to discuss the activity
(Assignee)

Comment 23

5 years ago
Sorry for that, my mind is asleep today......Forgot to say, another option is modify the flow so first we choose contacts from the list and THEN check for BT availability and pairing. This can be the functionality for 1.2 (much easier with current code), and maybe add the new activity for v1.3 ?
Joe, could you please include the right people from BT team in this thread?. Asking also Ayman for UX POV.
Flags: needinfo?(jcheng)
Flags: needinfo?(aymanmaat)
Hi Noemi, 

(In reply to Noemí Freire (:noemi) from comment #24)
> Joe, could you please include the right people from BT team in this thread?.
> Asking also Ayman for UX POV.

I'm in charge of Gecko Bluetooth development. I've been watching this bug since it was created, and I'll provide as much information as I can if necessary. Also, cc Ian, the owner of Gaia::Bluetooth File Transfer, to see if we will really hit the problem mentioned by Fernando in comment 22.
Flags: needinfo?(jcheng)

Updated

5 years ago
Depends on: 905606
Just adding bug 905606 dependency since in case of unpaired devices we need the activity returning to the export screen.
(Assignee)

Comment 27

5 years ago
As the bug 914112 has been closed as duplicate, updating the origin
Depends on: 910140
No longer depends on: 914112
Hi all,

Bug 905606 is in reviewing stage. After bug 905606 is landed, Bluetooth file transfer is ready to support inline pairing during sending files. So, I go through "FFOS_ContactsV1.2_20130805_V2.0.pdf" again, we have ability to meet the flow in "Send via Bluetooth "Single VCard" case. As long as Contacts::Export_Via_Bluetooth request 'share' activity with blob(package VCards in blob), Bluetooth File Transfer should be able to send the file out to a paired/unpaired device. If the device is unpaired, we're able to browse it in device list. After a user tapped the unpaired devices, Bluetooth File Transfer app will send the file while unpaired device is changed the state into paired. Please reference the spec. (https://bugzilla.mozilla.org/show_bug.cgi?id=892212#c13).

According to the spec. here, there are two things that can not be done yet.

* Send via Bluetooth : Multiple VCards : Step7, Page7
Once a user finished inline pairing in Bluetooth File Transfer app, we will send the files immediately. The step 7 "List of contacts" means that a user have ability to go back contacts app for picking contacts. Then, a user completed picking contacts to send file(VCard). The behavior is strange, and will make Bluetooth File Transfer program be interrupted in a "share" activity request.

* Notifications tray/overlay:
For such more information of transferring progress in detail, we'll need to have feasibility study for planing the implementation.

Thanks.
Just met Francisco and Fernando in workweek yesterday. They have already had a branch which is for exporting multiple contacts in a VCard via Bluetooth, but we faced different problems when we use different devices as receiving devices:

Case 1: The receiving side is a B2G device
Although the VCard transferring procedure could complete without any errors, when we tried to import VCard from the receiving side, an error screen "Can not open unknown media type" popped up.

Case 2: The receiving side is NOT a B2G device (e.g. HTC Sensation XL)
The receiving device would turn down the sending request immediately because the filename we sent has no extension.

I think both issues could be fixed from Gecko. I'll file another bug once I figure out the whole picture of the problem.

Updated

5 years ago
Flags: needinfo?(aymanmaat)
Removing Ayman ni since the BT flow for v1.2 has been clarified with BT team for v1.2 and a new discussion has been opened for v1.3.
(Assignee)

Comment 31

5 years ago
Created attachment 806049 [details] [review]
link to PR
Attachment #806049 - Flags: review?(fbsc)
Attachment #806049 - Flags: review?(fbsc) → review+
(Assignee)

Comment 32

5 years ago
merged on master (failing tests were unrelated): 53a8671f00abf23c637578becaf17e296515c02c

thanks to all for the help, specially Francisco for coding and Borja for big review.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Uplifted 53a8671f00abf23c637578becaf17e296515c02c to:
v1.2: 4d3cb5d0c9a491147c7ac7ce527bc7ebdc0ab31e
status-b2g-v1.2: --- → fixed
Attachment mime type: text/plain → text/x-github-pull-request
See Also: → bug 962707
You need to log in before you can comment on or make changes to this bug.