fix: add "Other" menu, for e.g. integration system links
This commit is contained in:
		
							parent
							
								
									811a37995d
								
							
						
					
					
						commit
						f4fdba1bbb
					
				
					 2 changed files with 20 additions and 1 deletions
				
			
		| 
						 | 
				
			
			@ -39,6 +39,7 @@ class SideshowMenuHandler(base.MenuHandler):
 | 
			
		|||
            self.make_customers_menu(request),
 | 
			
		||||
            self.make_products_menu(request),
 | 
			
		||||
            self.make_batch_menu(request),
 | 
			
		||||
            self.make_other_menu(request),
 | 
			
		||||
            self.make_admin_menu(request),
 | 
			
		||||
        ]
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -127,6 +128,16 @@ class SideshowMenuHandler(base.MenuHandler):
 | 
			
		|||
            ],
 | 
			
		||||
        }
 | 
			
		||||
 | 
			
		||||
    def make_other_menu(self, request, **kwargs):
 | 
			
		||||
        """
 | 
			
		||||
        Generate the "Other" menu.
 | 
			
		||||
        """
 | 
			
		||||
        return {
 | 
			
		||||
            'title': "Other",
 | 
			
		||||
            'type': 'menu',
 | 
			
		||||
            'items': [],
 | 
			
		||||
        }
 | 
			
		||||
 | 
			
		||||
    def make_admin_menu(self, request, **kwargs):
 | 
			
		||||
        """ """
 | 
			
		||||
        kwargs['include_people'] = True
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -9,4 +9,12 @@ class TestSideshowMenuHandler(WebTestCase):
 | 
			
		|||
    def test_make_menus(self):
 | 
			
		||||
        handler = mod.SideshowMenuHandler(self.config)
 | 
			
		||||
        menus = handler.make_menus(self.request)
 | 
			
		||||
        self.assertEqual(len(menus), 5)
 | 
			
		||||
        titles = [menu['title'] for menu in menus]
 | 
			
		||||
        self.assertEqual(titles, [
 | 
			
		||||
            'Orders',
 | 
			
		||||
            'Customers',
 | 
			
		||||
            'Products',
 | 
			
		||||
            'Batches',
 | 
			
		||||
            'Other',
 | 
			
		||||
            'Admin',
 | 
			
		||||
        ])
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue