Skip to content

Task/markets etag v7#459

Merged
greatest0fallt1me merged 2 commits into
Predictify-org:mainfrom
pchieneye:task/markets-etag-v7
Jul 25, 2026
Merged

Task/markets etag v7#459
greatest0fallt1me merged 2 commits into
Predictify-org:mainfrom
pchieneye:task/markets-etag-v7

Conversation

@pchieneye

Copy link
Copy Markdown

feat: add ETag support for /api/markets

Summary

This PR adds ETag support for the public markets endpoints so clients can perform conditional GET requests and receive 304 Not Modified responses when the response payload has not changed.

What changed

  • Added strong ETag generation and conditional revalidation handling for the public markets list and market detail responses.
  • Returned ETag and Cache-Control: no-cache headers for the relevant GET responses.
  • Added focused regression tests covering ETag headers and conditional revalidation behavior for the markets listing endpoint.
  • Documented the new caching/conditional request behavior in the markets API docs.

Closes: #387

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@pchieneye Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@greatest0fallt1me
greatest0fallt1me merged commit 3b0a464 into Predictify-org:main Jul 25, 2026
@greatest0fallt1me

Copy link
Copy Markdown
Contributor

Well-scoped PR — merging. Thanks for the contribution!

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.

Add ETag/304 on /api/markets (v7)

3 participants