diff --git a/lib/open_food_network/order_cycle_form_applicator.rb b/lib/open_food_network/order_cycle_form_applicator.rb
index 8f36a367ac..a5ed5fefea 100644
--- a/lib/open_food_network/order_cycle_form_applicator.rb
+++ b/lib/open_food_network/order_cycle_form_applicator.rb
@@ -105,17 +105,15 @@ module OpenFoodNetwork
       @user_manages_coordinator = Enterprise.managed_by(@spree_current_user).include? @order_cycle.coordinator
     end
 
-    # TODO Need to use editable rather than visible
     def editable_variant_ids_for_incoming_exchange_between(sender, receiver)
       OpenFoodNetwork::Permissions.new(@spree_current_user).
-        visible_variants_for_incoming_exchanges_between(sender, receiver, order_cycle: @order_cycle).
+        editable_variants_for_incoming_exchanges_between(sender, receiver, order_cycle: @order_cycle).
         pluck(:id)
     end
 
-    # TODO Need to use editable rather than visible
     def editable_variant_ids_for_outgoing_exchange_between(sender, receiver)
       OpenFoodNetwork::Permissions.new(@spree_current_user).
-      visible_variants_for_outgoing_exchanges_between(sender, receiver, order_cycle: @order_cycle).
+      editable_variants_for_outgoing_exchanges_between(sender, receiver, order_cycle: @order_cycle).
       pluck(:id)
     end