ProtonMail / proton-contacts

React web application to manage ProtonMail contacts
54 stars 19 forks source link

Manage prefs properly #343

Closed econdepe closed 4 years ago

econdepe commented 4 years ago

For Vcard fields that admit more than one value, there is a PREF parameter that sets the preference of the different values. We were not saving this parameter in the Vcard, leading to the bugs described in #341 and #339.

Fixes #341. Fixes #339

As a sidenote: this bug went undetected till now because somehow the fields were being saved in the right order in the Vcard (without the PREF parameter though), and read back in the same order. That order in the read-back was however altered in Firefox.