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: allow local evaluation #79

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

duyhungtnn
Copy link
Collaborator

@duyhungtnn duyhungtnn commented Oct 26, 2024

This PR integrates a Node evaluation library and enables the local evaluation feature with in-memory caching.

@duyhungtnn duyhungtnn force-pushed the feat/evaluation-module branch from f44c2ec to a398def Compare October 26, 2024 04:03
@duyhungtnn duyhungtnn force-pushed the feat/evaluation-module branch 4 times, most recently from 18fcff9 to 6c15f34 Compare November 23, 2024 14:42
@duyhungtnn duyhungtnn marked this pull request as ready for review November 25, 2024 09:49
@duyhungtnn duyhungtnn force-pushed the feat/evaluation-module branch from 321bf84 to 94916ff Compare December 2, 2024 15:45
fix: missing e2e tests

fix: invalid API Key for test e2e

test: update e2e test for event

fix: e2e test fail

fix: typo error with segment user

chore: remove BKTClientImpl from index.js

feat: allow local evaluation
@duyhungtnn duyhungtnn force-pushed the feat/evaluation-module branch from d165dd4 to c34970e Compare December 10, 2024 13:18
@duyhungtnn
Copy link
Collaborator Author

Updated:

  • Use @bucketeer/evaluation from npm
  • Skip check the feature version for the local evaluation e2e tests
  • Improvement error handling with the local evaluation

@duyhungtnn
Copy link
Collaborator Author

@cre8ivejp I fixed the conflict. Please help me take a look

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