-
Notifications
You must be signed in to change notification settings - Fork 20
Autocomplete addresses from Contacts #8
Comments
It would be awesome if the same could be done between this app and the ldap backend (it'd obviously need some reworking to make that data available). Are we setting milestones for OC 10 yet? 😄 |
Well, this here is about the Contacts app for now, not about the user management. Cause there we don’t even have addresses. |
I wrote a piece of code, which filters contact names and their addresses based on the search term. But I won't create a PR, since this is currently not good code and there are still peding PRs |
@v1r0x yeah, I’ll hope to get to it these days. :) The ownCloud conf was very busy. Hope you can make it next year, probably end of August again! https://owncloud.org/conf |
@jancborchardt hopefully! There should be no thesis next year ;) |
@v1r0x if you just always branch PRs off master then it should be fine. :) |
@jancborchardt I know, but I continued master on my productive setup (I know this is not clever :D) and thus I have all my PRs already included. Will separate them after my holiday. |
@v1r0x hehe, yeah it’s indeed not clever. ;) Master should never be worked on, only in branches which can then be opened as pull requests. |
And yeah, see you later! :) |
When typing »Sander« into the search field (and assuming I have your address in ownCloud Contacts) I should get the suggestion »Sander, Somestreet, Somecity, Netherlands«.
This would be a great example of inter-app connection in ownCloud. If you want to see how it’s done, check out the mail app code: https://github.com/owncloud/mail (we do that for the »to« and »cc« fields)
The text was updated successfully, but these errors were encountered: