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

Use Cached Endpoints #281

Closed
wants to merge 2 commits into from
Closed

Use Cached Endpoints #281

wants to merge 2 commits into from

Conversation

mkouzel-yext
Copy link
Contributor

This change updated the LiveAPI endpoints that search-core provides so that it provides the cached version of the endpoints. Using the cached endpoints helps improve performance.

J=WAT-4442
TEST=manual

Built test site with new change, saw that the cached endpoints were being used.

This change updated the LiveAPI endpoints that search-core provides
so that it provides the cached version of the endpoints. Using the
cached endpoints helps improve performance.

J=WAT-4442
TEST=manual

Built test site with new change, saw that the cached endpoints
were being used.
@mkouzel-yext mkouzel-yext requested a review from a team as a code owner November 12, 2024 18:38
@Fondryext
Copy link
Contributor

So the way we usually make changes to our github repos is:

  1. Make a new branch for the new version. In this case this'll be 2.5.5, so the branch will probably be called hotfix/v2.5.5
  2. Up the package version on that branch correspondingly. In a lot of our repos this change gets created automatically
  3. Make a pull request to merge your changes into the hotfix branch
  4. Once all changes are in the hotfix branch, then make a request from the hoftix branch into master
  5. Once that's pulled, we make a new release tag and publish to npm, and possibly pull into develop

Happy to expand on any of these

Increments the package version from v2.5.4 to v2.5.5 for this hotfix.
@mkouzel-yext mkouzel-yext deleted the dev/use-cached-endpoints branch November 12, 2024 19:22
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.

2 participants