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

Support multiple type declaration #942

Closed
1 of 2 tasks
skjnldsv opened this issue Feb 19, 2019 · 17 comments
Closed
1 of 2 tasks

Support multiple type declaration #942

skjnldsv opened this issue Feb 19, 2019 · 17 comments
Labels
4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working high High priority

Comments

@skjnldsv
Copy link
Member

skjnldsv commented Feb 19, 2019

Looking at the rfc: https://tools.ietf.org/html/rfc6350#section-6.4.1

 The default type is "voice".  These type parameter values can be
 specified as a parameter list (e.g., TYPE=text;TYPE=voice) or as a
 value list (e.g., TYPE="text,voice").  The default can be
 overridden to another set of values by specifying one or more
 alternate values.  For example, the default TYPE of "voice" can be
 reset to a VOICE and FAX telephone number by the value list
 TYPE="voice,fax".

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@skjnldsv skjnldsv added bug Something isn't working 1. to develop Accepted and waiting to be taken care of high High priority blocked Waiting for a fix on one of our dependencies labels Feb 19, 2019
@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #414 (Browser Support), #518 (CardDav types), #679 (Full CLI Support), #608 (Org vcard support), and #609 (GROUP vcard support).

@jellium
Copy link

jellium commented Jul 9, 2019

Hello, I understand this issue is related to the way iOS handles wrongly vCard types when set as a value list (e.g. doesn't handle the syntax like TYPE="VOICE,CELL" as defined by the standard but accepts TYPE=VOICE,CELL instead). However, is there any workaround for Nextcloud × iOS users who would accept to have the Nextcloud vcf file not formatted properly (i.e. with types as a value list but without quotes)? I tried to manually import, remove quotes and export but this doesn't work.
Thanks a lot.

@Nemental
Copy link

Something new in here?

@skjnldsv
Copy link
Member Author

skjnldsv commented Oct 11, 2019

@Nemental still waiting for kewisch/ical.js#388

@skjnldsv
Copy link
Member Author

FIX HAVE BEEN MERGED 🎉

@skjnldsv skjnldsv added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Jan 13, 2020
@skjnldsv
Copy link
Member Author

Waiting for new ical.js release now

@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress blocked Waiting for a fix on one of our dependencies labels Apr 1, 2020
@skjnldsv
Copy link
Member Author

skjnldsv commented Apr 1, 2020

Next contacts release will fix this :D
Mozilla released ical.js v1.4.0 today! 🎉 🚀

@skjnldsv
Copy link
Member Author

Please help test the new release (link in first post): #1579

@apastuszak
Copy link

I don't think this is fixed. I'm running Contacts 3.3.0 on Nextcloud 18.0.4 and this is what I see with my Nextcloud contacts:

icloud_contact

Note how the home number still shows "HOME instead of home.

@veritas06
Copy link

I'm in the same boat as apastuszak. Tried editing an existing contact & even creating a new one, but still seeing the same issue. I can run through & document all the steps again, but still has not been working for me.

Nextcloud 18.0.3
Contacts 3.3.0

@apastuszak
Copy link

apastuszak commented May 18, 2020

Here is what I am seeing. If I create a new contact in Nextcloud and then iOS syncs it, and I export it as vcf, I see this line:

TEL;TYPE="HOME,VOICE":(xxx) xxx-xxxx

And the iPhone displays "HOME

If I then edit the contact and change the phone type to home on the iPhone, the vcf export from Nextcloud shows this:

TEL;type=HOME;type=VOICE;type=pref:(xxx) xxx-xxxx

Either vcf format displays correctly in Nextcloud.

@skjnldsv
Copy link
Member Author

skjnldsv commented Jun 2, 2020

Yep, the issue is the quote now.
Multiple types are different.

Also, this is not released as far as I recall

@nsrafk
Copy link

nsrafk commented Oct 16, 2020

How is this progressing? I really cant migrate away from iCloud contacts until this works :-)

Good job everybody - sadly i can’t contribute with :[

@F3ELING1337
Copy link

Yes I would also like to know how the status of this is? Currently on NC 20.0.2 and iOS 14.2:
Contacts on iOS synced with NC still show things like "CELL or "HOME instead of 'valid' values.
Thanks for all your help to the developers!

@veritas06
Copy link

Any update on this? I see the issue is closed but it's still not resolved.

@veritas06
Copy link

Can we get this re-opened since the issue is not resolved?

@tamburra
Copy link

tamburra commented Feb 5, 2021

This has been an issue for a long time and still is with all current releases as of Feb 5, 2021. Would love for this to get reopened and possibly resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working high High priority
Projects
None yet
Development

No branches or pull requests

9 participants