Merge pull request #2043 from kili-technology/feature/lab-4295-aau-i-add-update-and-delete-steps-with-the-sdk-3

fix(LAB-4295): use the name of the step instead of id in add_review_s…

Author: stchabi

src/kili/adapters/kili_api_gateway/project_workflow/operations_mixin.py

@@ -203,6 +203,13 @@ def add_review_step(self, data: AddReviewStepInput) -> dict:
         existing_members = self.list_activated_project_users(data.project_id)
         assignees_to_add = get_assignees_to_add_ids(existing_members, data.assignees)
         data.assignees = assignees_to_add
+        steps = self.get_steps(data.project_id, fields=["steps.id", "steps.name"])
+        send_back_to_step = next(
+            (step.get("id") for step in steps if step.get("name") == data.send_back_to_step), None
+        )
+        if not send_back_to_step:
+            raise ValueError("The sendBackToStep name given does not exist")
+        data.send_back_to_step = send_back_to_step
         variables = {"input": add_review_step_input_mapper(data)}
         mutation = get_add_review_step_mutation()
         result = self.graphql_client.execute(mutation, variables)

src/kili/presentation/client/project_workflow.py

@@ -163,7 +163,7 @@ def add_review_step(
             assignees: List of user emails to assign as reviewers.
             step_coverage: Percentage of assets to be reviewed in this step (0-100).
             use_honeypot: Whether to use honeypot on this step.
-            send_back_to_step: Id of the step to send assets back to.
+            send_back_to_step: Name of the step to send assets back to.
 
         Returns:
             A dict with the created step data (id, name).