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.

Sign up for GitHub

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

Jump to bottom

add caching and request header #1760

Closed
wants to merge 8 commits into from
Closed

add caching and request header #1760

wants to merge 8 commits into from

Conversation

vairafiq
Copy link
Member

@vairafiq vairafiq commented Jun 5, 2024

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

How to reproduce the issue or how to test the changes

Issue: Directorist.com is experiencing more API request

  • Qnique header added so that remote server can track and whitelist the APIs
  • Cached promo products for 1 hour

Any linked issues

Fixes #

Checklist

@vairafiq vairafiq requested a review from syedgalib as a code owner June 5, 2024 10:43
@vairafiq vairafiq added this to the v7.10.4 milestone Jun 27, 2024
@vairafiq
Copy link
Member Author

Closed and continue to #1766

@vairafiq vairafiq closed this Jun 27, 2024
@vairafiq vairafiq deleted the improve/licensing branch June 27, 2024 05:24
@vairafiq vairafiq removed this from the v7.10.4 milestone Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@syedgalib syedgalib Awaiting requested review from syedgalib syedgalib is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vairafiq @obiPlabon