891385 - [Contacts] Add "org" field to indexed contact fields

Status: RESOLVED WONTFIX

(Core Graveyard :: DOM: Contacts, defect, P5)

Description

[Rick Waldron [:rwaldron]]

Needed for adding "search by org" capabilities to contact lookups

See also https://bugzilla.mozilla.org/show_bug.cgi?id=890706

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Note that searching on 'org' and 'tel' is implemented by the contacts app by performing the search within the app instead of using the ContactsAPI search functionality.

Just FYI for comparison.  Its unclear to me which is a better solution.

Have you asked about this on the dev-webapi list?

Comment 2

[Julien Wajsberg [:julienw]]

The contacts app is using the DOM to search. It's really a bad solution (just try to search before everything is loaded).

Eventually, in a datastore-enabled API, we'll need to do differently. So clearly there is no need to rush here, maybe we'll do it differently in 1.2 if we switch to datastore anyway.

Comment 3

[Ben Kelly [:bkelly, not reviewing]]

(In reply to Julien Wajsberg [:julienw] (PTO 15th -> 28th July) from comment #2)
> The contacts app is using the DOM to search. It's really a bad solution
> (just try to search before everything is loaded).

This particular problem should be improved in master for contacts app.  I refactored some things while integrating visibility monitor to allow you to start searching while the load is in progress.  It does take a long time if you are searching for the last person to load, though.

> Eventually, in a datastore-enabled API, we'll need to do differently. So
> clearly there is no need to rush here, maybe we'll do it differently in 1.2
> if we switch to datastore anyway.

Yea, I guess I was curious if you had heard anything about this lately from the webapi folks.  My impression was there was resistance to adding more indexes to ContactsAPI in favor of using a client-side solution like datastore.

Comment 4

[Julien Wajsberg [:julienw]]

(In reply to Ben Kelly [:bkelly] from comment #3)
> 
> Yea, I guess I was curious if you had heard anything about this lately from
> the webapi folks.  My impression was there was resistance to adding more
> indexes to ContactsAPI in favor of using a client-side solution like
> datastore.

yep, same here, but this is a big task...

Comment 5

[Preeti Raghunath(:Preeti)]

Seems like a feature request, hence minus.

Comment 6

[Firefox Bug Husbandry Bot]

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

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.

Comment 7

[Sylvestre Ledru [:Sylvestre]]

DOM: Contacts isn't used anymore. 
Closing all remaining bugs.