Skip to content

Improved vcard photo handling - #19901

Merged
rullzer merged 4 commits into
masterfrom
bugfix/noid/vcard-photo-handling
Apr 17, 2020
Merged

Improved vcard photo handling#19901
rullzer merged 4 commits into
masterfrom
bugfix/noid/vcard-photo-handling

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

No description provided.

Comment threadapps/dav/lib/CardDAV/CardDavBackend.php
Comment threadapps/dav/lib/CardDAV/CardDavBackend.php
Comment threadapps/dav/lib/CardDAV/CardDavBackend.php Outdated

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
🐘

@rullzer

Copy link
Copy Markdown
Member

CI says nope

@nickvergessen
nickvergessenforce-pushed the bugfix/noid/vcard-photo-handling branch from ff0b444 to fabec4cCompareApril 2, 2020 08:47
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

@georgehrke final +1?

Comment threadapps/dav/lib/CardDAV/CardDavBackend.php
Comment threadapps/dav/lib/CardDAV/CardDavBackend.php Outdated
This was referenced Apr 4, 2020
@rullzer

Copy link
Copy Markdown
Member

sooooo?

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

I'm currently unsure about @georgehrke s comments.
Should we generate a new random etag?

nickvergessenand others added 4 commits April 15, 2020 15:49
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
@georgehrke
georgehrkeforce-pushed the bugfix/noid/vcard-photo-handling branch from fabec4c to fe4527aCompareApril 15, 2020 13:57
This was referenced Apr 15, 2020
@rullzer

Copy link
Copy Markdown
Member

Good to go now?

@georgehrke

Copy link
Copy Markdown
Member

👍 from my side

@rullzer
rullzer merged commit ed56619 into masterApr 17, 2020
@rullzer
rullzer deleted the bugfix/noid/vcard-photo-handling branch April 17, 2020 09:43
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: dav

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nickvergessen@rullzer@georgehrke@skjnldsv