Skip to content

Conversation

@kobylynskyi
Copy link
Owner

@joffrey-bion, @lmartellotto, @logicatmidod, could you please review? Thanks!


Closes #340

@kobylynskyi kobylynskyi self-assigned this Oct 9, 2020
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 1 Code Smell

81.5% 81.5% Coverage
0.0% 0.0% Duplication

Copy link

@logicatmidod logicatmidod left a comment

Choose a reason for hiding this comment

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

This will work I have tried at my end by replicating the code. But need to check if some how response wrapper also can be created for managing these responses. Though we should be providing ability to register Converters to those responses. That can convert to the appropriate Objects.

@kobylynskyi kobylynskyi merged commit 3fb7bce into master Oct 28, 2020
@kobylynskyi kobylynskyi deleted the 340-multiple-queries-in-request branch October 28, 2020 21:39
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.

Combining Multiple Queries in Request

3 participants