Skip to content

Conversation

@ThomasLandauer
Copy link
Member

  • I created a new header "JSONPath", and moved the general JSONPath infos from grabDataFromResponseByJsonPath to it - in preparation for removing flow/jsonpath as a dependency, see Abandoned dependency #24 (comment)
  • I removed the "Parts" header, cause it seemed useless to me

* I created a new header "JSONPath", and moved the *general* JSONPath infos from `grabDataFromResponseByJsonPath` to it - in preparation for removing `flow/jsonpath` as a dependency, see Codeception#24 (comment)
* I removed the "Parts" header, cause it seemed useless to me
@Naktibalda
Copy link
Member

Parts header is not useless, it could be clearer, but, I think, the assumption is that the reader is familiar with parts functionality from some other documentation page.

part: Json parameter to REST module in suite configuration can be used to import only Json methods and part: Xml - only Xml methods.

See [Part] line under each method in https://codeception.com/docs/modules/REST

Updating JsonPath, according to Codeception#36
Bringing back the parts, according to Codeception/Codeception#6026
@ThomasLandauer
Copy link
Member Author

This is ready for merging now.

Question: What's the reason for having those parts in this module? Is there any other module that conflicts with the entire REST module?

@Naktibalda Naktibalda merged commit 4b1f599 into Codeception:master Apr 5, 2021
@ThomasLandauer ThomasLandauer deleted the patch-2 branch April 5, 2021 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants