Skip to content
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

feat: Cursored APIResponse #74

Closed
wants to merge 2 commits into from
Closed

Conversation

PhilippeWeidmann
Copy link
Member

No description provided.

@PhilippeWeidmann PhilippeWeidmann requested a review from a team as a code owner October 20, 2023 11:34
@PhilippeWeidmann PhilippeWeidmann marked this pull request as draft October 20, 2023 11:56
@PhilippeWeidmann PhilippeWeidmann force-pushed the cursored-api-response branch 2 times, most recently from d95bc01 to 782d650 Compare November 3, 2023 13:24
@PhilippeWeidmann PhilippeWeidmann force-pushed the cursored-api-response branch 4 times, most recently from cb2dc05 to bf243e8 Compare November 9, 2023 11:45
@adrien-coye
Copy link
Contributor

@PhilippeWeidmann Is this PR going to break compatibility, or is it backward compatible?

@PhilippeWeidmann
Copy link
Member Author

It is breaking, we move from responseAt to ApiResponse in the return

Copy link

sonarcloud bot commented Dec 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link

sonarcloud bot commented Jan 29, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@valentinperignon valentinperignon removed the request for review from a team March 1, 2024 10:46
@PhilippeWeidmann PhilippeWeidmann deleted the cursored-api-response branch March 21, 2024 15:01
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.

None yet

2 participants