Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unified Search API Response Update For Contact #3997

Closed
alperozturk96 opened this issue Jun 24, 2024 · 0 comments · Fixed by nextcloud/server#46464
Closed

Unified Search API Response Update For Contact #3997

alperozturk96 opened this issue Jun 24, 2024 · 0 comments · Fixed by nextcloud/server#46464
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature or request

Comments

@alperozturk96
Copy link

alperozturk96 commented Jun 24, 2024

We want to open contact directly in mobile application rather than redirect to web app. Therefore, we need to use Unified Search API to get contact and try to match with already synced internal contacts by Dav5x. So that user can open contact in phone.

Dav5x contact ID and internal Android contact ID is not matching so we need to provide uniqueness via depending on DISPLAY_NAME, EMAIL, PHONE_NUMBER or UID (if we have on both side even better)

VCARD

Current Implementation

{
"thumbnailUrl": "",
"title": "Test",
"subline": "[dummy@test.de](https://cloud.nextcloud.com/call/mailto:dummy@test.de)",
"resourceUrl": "http://localhost/nc/index.php/apps/contacts/direct/contact/8ccc7b7f-19ee-4f9c-9a70-ab78cfc9ac0c~contacts",
"icon": "icon-contacts-dark",
"rounded": true,
"attributes": []
}

Suggestion

{
"thumbnailUrl": "",
"title": "Test",
"subline": "[dummy@test.de](https://cloud.nextcloud.com/call/mailto:dummy@test.de)",
"resourceUrl": "http://localhost/nc/index.php/apps/contacts/direct/contact/8ccc7b7f-19ee-4f9c-9a70-ab78cfc9ac0c~contacts",
"icon": "icon-contacts-dark",
"rounded": true,
"attributes": [
"displayName": "Arthur Kirkland"
"email": "example@mail.com"
"phoneNumber": 00000000000
]
}

??? Better Alternative Need To Check For Android ???

{
"thumbnailUrl": "",
"title": "Test",
"subline": "[dummy@test.de](https://cloud.nextcloud.com/call/mailto:dummy@test.de)",
"resourceUrl": "http://localhost/nc/index.php/apps/contacts/direct/contact/8ccc7b7f-19ee-4f9c-9a70-ab78cfc9ac0c~contacts",
"icon": "icon-contacts-dark",
"rounded": true,
"attributes": [
"uid": "4fbe8971-0bc3-424c-9c26-36c3e1eff6b1"
]
}
@alperozturk96 alperozturk96 added enhancement New feature or request 0. Needs triage labels Jun 24, 2024
@hamza221 hamza221 added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage labels Jul 1, 2024
tobiasKaminsky added a commit to nextcloud/server that referenced this issue Jul 12, 2024
fix nextcloud/contacts#3997

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
tobiasKaminsky added a commit to nextcloud/server that referenced this issue Jul 16, 2024
fix nextcloud/contacts#3997

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
tobiasKaminsky added a commit to nextcloud/server that referenced this issue Jul 24, 2024
fix nextcloud/contacts#3997

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
tobiasKaminsky added a commit to nextcloud/server that referenced this issue Jul 24, 2024
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Altahrim pushed a commit to nextcloud/server that referenced this issue Jul 25, 2024
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants