Add clarifying comments

Author: toupper

WooCommerce/Classes/ViewRelated/Dashboard/Settings/In-Person Payments/InPersonPaymentsMenuViewController.swift

@@ -126,6 +126,8 @@ private extension InPersonPaymentsMenuViewController {
             return
         }
 
+        // Instead of using `CardPresentPaymentsOnboardingPresenter` we create the view directly because we already have the onboarding state in the use case.
+        // That way we avoid triggering the onboarding check again that comes with the presenter.
         let onboardingViewModel = InPersonPaymentsViewModel(useCase: cardPresentPaymentsOnboardingUseCase, showMenuOnCompletion: false)
         onboardingViewModel.onOnboardingCompletion = { [weak self] plugin in
             self?.refreshAfterNewOnboardingState(.completed(plugin: plugin))

WooCommerce/Classes/ViewRelated/Orders/Collect Payments/CardPresentPaymentsOnboardingPresenter.swift

@@ -9,6 +9,9 @@ protocol CardPresentPaymentsOnboardingPresenting {
     func refresh()
 }
 
+/// Checks for the current user status regarding Card Present Payments,
+/// and shows the onboarding if the user didn't finish the onboarding to use CPP
+/// 
 final class CardPresentPaymentsOnboardingPresenter: CardPresentPaymentsOnboardingPresenting {
 
     private let stores: StoresManager