-
Notifications
You must be signed in to change notification settings - Fork 24
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
Revamp the Contacts UI in preparation for the DAV migration #96
Conversation
Not editable as of now
Show a link to old contacts interface instead
Emails are visible by default, the rest are not
Personal is more fitting in some situations (re emails for example), but that's the way nextcloud is doing this, and is generally closer to what vCard itself specifies.
- fixed routing to not show header toolbar in login screen - support both legacy api with one email addr per contact and new with multiple email addresses per contact - support both numeric and string contact id
…ompatibility to contact.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tadzik Would it be possible for you to rebase this on top of master, and reduce the number of commits to one ( or the number of commits that makes sense to show in the public git history )?
Let me know if you need some tips about how to reduce the number of commits, and remove merge commits etc.
This has been squashed into #102, so closing this one. |
No description provided.