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

adding swatStatistics file to make statistics.json call to apps.bazaarvoice.com and fix for PD-243471 #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

VarshaAdigaBV
Copy link
Collaborator

@VarshaAdigaBV VarshaAdigaBV commented Jun 12, 2024

PR description

JIRA tickets

PD-240923
PD-243471

This PR is for

  • Feature

Requirements / issues

  • Make statistics.json calls to apps.bazaarvoice.com when swat BFD flag is enabled.
  • Resolve Cannot read properties of undefined (reading 'headers') error by bvFetch.

Solutions

  • Created a new function to be used by inline ratings to make statistics.json calls to apps.bazaarvoice.com when swat BFD flag is enabled.
  • Added logic to check if cachedResponse if present in the cache.

Check points

https://bazaarvoice.atlassian.net/wiki/spaces/DEV/pages/78853865680/Design+to+Change+API+endpoint+to+apps.bazaarvoice.com+for+v2+hosted+display+submission+applications

  • Functionality

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.

1 participant