Merge pull request #267 from spark-solutions/fix/spree-3-x-admin-menu

Makes sidebar menu deface override work with both Spree 3.x and 4.x
spree-contrib · Sep 19, 2019 · 5fbc990 · 5fbc990
2 parents 15e1fa8 + e4b9e49
commit 5fbc990
Showing 1 changed file with 15 additions and 6 deletions.
diff --git a/app/overrides/add_pages_to_admin_sidebar_menu.rb b/app/overrides/add_pages_to_admin_sidebar_menu.rb
@@ -1,6 +1,15 @@
-Deface::Override.new(
-  virtual_path: 'spree/admin/shared/_main_menu',
-  name: 'pages_admin_sidebar_menu',
-  insert_bottom: 'nav',
-  partial: 'spree/admin/shared/pages_sidebar_menu'
-)
+if Spree.version.to_f < 4.0
+  Deface::Override.new(
+    virtual_path: 'spree/layouts/admin',
+    name: 'pages_admin_sidebar_menu',
+    insert_bottom: '#main-sidebar',
+    partial: 'spree/admin/shared/pages_sidebar_menu'
+  )
+else
+  Deface::Override.new(
+    virtual_path: 'spree/admin/shared/_main_menu',
+    name: 'pages_admin_sidebar_menu',
+    insert_bottom: 'nav',
+    partial: 'spree/admin/shared/pages_sidebar_menu'
+  )
+end