Implement suggested comments

Author: esrakartalOpt

optimizely/decision_service.py

@@ -672,7 +672,7 @@ def get_variation_for_feature(
             - 'error': Boolean indicating if an error occurred during the decision process.
             - 'reasons': List of log messages representing decision making for the feature.
         """
-        holdouts = project_config.get_holdouts_for_flag(feature.id)
+        holdouts = project_config.get_holdouts_for_flag(feature.key)
 
         if holdouts:
             # Has holdouts - use get_decision_for_flag which checks holdouts first
@@ -708,7 +708,7 @@ def get_decision_for_flag(
         user_id = user_context.user_id
 
         # Check holdouts
-        holdouts = project_config.get_holdouts_for_flag(feature_flag.id)
+        holdouts = project_config.get_holdouts_for_flag(feature_flag.key)
         for holdout in holdouts:
             holdout_decision = self.get_variation_for_holdout(holdout, user_context, project_config)
             reasons.extend(holdout_decision['reasons'])

optimizely/project_config.py

@@ -834,19 +834,19 @@ def get_flag_variation(
 
         return None
 
-    def get_holdouts_for_flag(self, flag_id: str) -> list[HoldoutDict]:
+    def get_holdouts_for_flag(self, flag_key: str) -> list[HoldoutDict]:
         """ Helper method to get holdouts from an applied feature flag.
 
         Args:
-            flag_id: (REQUIRED) ID of the feature flag.
+            flag_key: Key of the feature flag.
 
         Returns:
             The holdouts that apply for a specific flag.
         """
         if not self.holdouts:
             return []
 
-        return self.flag_holdouts_map.get(flag_id, [])
+        return self.flag_holdouts_map.get(flag_key, [])
 
     def get_holdout(self, holdout_id: str) -> Optional[HoldoutDict]:
         """ Helper method to get holdout from holdout ID.