Skip to content

Added functions to check etag of elements#26370

Merged
SergioBertolinSG merged 3 commits into
masterfrom
integration_tests_adding_etag_support
Oct 14, 2016
Merged

Added functions to check etag of elements#26370
SergioBertolinSG merged 3 commits into
masterfrom
integration_tests_adding_etag_support

Conversation

@SergioBertolinSG

Copy link
Copy Markdown
Contributor

Improvements to play with etags.

@SergioBertolinSG SergioBertolinSG added this to the 9.2 milestone Oct 13, 2016
@mention-bot

Copy link
Copy Markdown

@SergioBertolinSG, thanks for your PR! By analyzing the history of the files in this pull request, we identified @PVince81, @rullzer and @LukasReschke to be potential reviewers.

private $usingOldDavPath = true;
/** @var ResponseInterface */
private $response;
/** @var array */

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"map with path as key and etag as value"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually it is a map with user as key an another map as value, which has path as key and etag as value.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changed.

@PVince81

Copy link
Copy Markdown
Contributor

Adjust the comment then it's good 👍

@PVince81

Copy link
Copy Markdown
Contributor

👍 please merge if you're done. (label still says "2 - Developing")

@lock

lock Bot commented Aug 4, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants