-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Write profile properties and backend properties to system address book #37800
Comments
It looks like role and organization are among the props intentionally not published: server/apps/settings/src/constants/AccountPropertyConstants.js Lines 148 to 154 in 74f31ba
@Pytal do you recall the details of this? Do we just not want the four props to ever hit the lookup server? I'd like to use them in the system address book. This is also related to #19575 (comment). I hope that we can store more local properties in the system addressbook and perhaps filter when we send data to other server? |
@nickvergessen says these props were not meant for publishing but only the profile. So two options here
|
Can't be super certain but iirc as those 4 new properties were only for profile they were not added to the list of properties to be published alongside the ones in https://github.com/nextcloud/server/blob/d6847a975767bb610018d531d614b50f1da09a98/apps/dav/lib/CardDAV/Converter.php#L59C1-L95 |
Option 3: We will address #19575 (comment) because it has wider issues with missing local data in the system address book. So once that is fixed, the private or local role and organization will be mappable. We don't have to make them published/federated. |
We noticed that language is not a profile property. It is a simple user setting shown on the profile settings page. @jancborchardt do we still want to write a user's configured language as their LANG prop of the system address book contact entry? If so, under what scope? Can we assume everyone is okay with others seeing their language? Semantically it is tricky to interpret the number of the LANG property because it's not necessarily the user's spoken language but what they configure Nextcloud to be in. I've set my Nextcloud to English while I speak also German. |
Part of #19575
Part 1: read from user backends
Part 2: write to VCARD
Mapping
Language -> LANGhttps://cloud.domain.tld/u/<uid>
(if enabled) -> URL Write profile URL into system address book contact #38944https://cloud.domain.tld/apps/spreed/?callUser=<uid>
-> URLThe text was updated successfully, but these errors were encountered: