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

fix: change GetEvaluations request field #461

Merged
merged 3 commits into from
Jul 12, 2023
Merged

fix: change GetEvaluations request field #461

merged 3 commits into from
Jul 12, 2023

Conversation

masaaania
Copy link
Contributor

@masaaania masaaania commented Jul 12, 2023

  • This PR adds the user_evaluation_condition field to GetEvaluationsRequest to determine which evaluation method should be used.
    • If the user_evaluation_condition field is not set to any value, Bucketeer evaluates the flags with the old evaluation method.

@masaaania masaaania marked this pull request as ready for review July 12, 2023 05:54
@masaaania masaaania requested a review from cre8ivejp July 12, 2023 05:54
Comment on lines -38 to +42
int64 evaluated_at = 6;
bool user_attributes_updated = 7;
UserEvaluationCondition user_evaluation_condition = 6;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any SDK does not use the evaluated_at and user_attributes_updated.

@masaaania
Copy link
Contributor Author

@cre8ivejp
I've moved UserEvaluationCondition from feature/evaluation.proto to gateway/service.proto.

Copy link
Member

@cre8ivejp cre8ivejp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@masaaania masaaania merged commit d98e147 into main Jul 12, 2023
@masaaania masaaania deleted the get_evaluations branch July 12, 2023 09:46
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