From 956bb1f1b4d92de5c77217f30f172d793f2355f3 Mon Sep 17 00:00:00 2001
From: Alex Dusenbery <adusenbery@edx.org>
Date: Thu, 6 Aug 2020 11:52:58 -0400
Subject: [PATCH] Enterprise users for which the learner portal is enabled
 should not see 'Order History' in their user menu, since it is not relevant
 to their method of purchasing content.

---
 lms/templates/header/user_dropdown.html | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/lms/templates/header/user_dropdown.html b/lms/templates/header/user_dropdown.html
index 79c213f025..84bfc6aa27 100644
--- a/lms/templates/header/user_dropdown.html
+++ b/lms/templates/header/user_dropdown.html
@@ -23,6 +23,9 @@ username = self.real_user.username
 resume_block = retrieve_last_sitewide_block_completed(self.real_user)
 displayname = get_enterprise_learner_generic_name(request) or username
 enterprise_customer_portals = get_enterprise_learner_portals(request)
+## Enterprises with the learner portal enabled should not show order history, as it does
+## not apply to the learner's method of purchasing content.
+should_show_order_history = should_redirect_to_order_history_microfrontend() and not enterprise_customer_portals
 %>
 
 <div class="nav-item hidden-mobile">
@@ -50,7 +53,7 @@ enterprise_customer_portals = get_enterprise_learner_portals(request)
 
         <div class="mobile-nav-item dropdown-item dropdown-nav-item"><a href="${reverse('learner_profile', kwargs={'username': username})}" role="menuitem">${_("Profile")}</a></div>
         <div class="mobile-nav-item dropdown-item dropdown-nav-item"><a href="${reverse('account_settings')}" role="menuitem">${_("Account")}</a></div>
-        % if should_redirect_to_order_history_microfrontend():
+        % if should_show_order_history:
             <div class="mobile-nav-item dropdown-item dropdown-nav-item"><a href="${settings.ORDER_HISTORY_MICROFRONTEND_URL}" role="menuitem">${_("Order History")}</a></div>
         % endif
         <div class="mobile-nav-item dropdown-item dropdown-nav-item"><a href="${reverse('logout')}" role="menuitem">${_("Sign Out")}</a></div>