Remove category dropdown from pick list creation
This commit is contained in:
@@ -13,8 +13,7 @@ test.describe('Active pick list', () => {
|
||||
await page.getByRole('link', { name: 'Create Pick List' }).click();
|
||||
await page.getByLabel('Area').click();
|
||||
await page.getByRole('option', { name: areaName }).first().click();
|
||||
await page.getByLabel('Category (optional)').click();
|
||||
await page.getByRole('option', { name: 'Chocolates' }).click();
|
||||
await page.getByRole('checkbox', { name: 'Chocolates' }).click();
|
||||
await page.getByRole('checkbox', { name: 'Chips' }).click();
|
||||
await page.getByRole('button', { name: 'Save Pick List' }).click();
|
||||
|
||||
|
||||
@@ -97,7 +97,6 @@ describe('StartPickListScreen', () => {
|
||||
</MemoryRouter>,
|
||||
);
|
||||
|
||||
expect(screen.getByLabelText(/category \(optional\)/i)).toBeVisible();
|
||||
categoriesMock.forEach((category) => {
|
||||
expect(screen.getByRole('checkbox', { name: category.name })).toBeInTheDocument();
|
||||
});
|
||||
@@ -115,9 +114,7 @@ describe('StartPickListScreen', () => {
|
||||
await user.click(screen.getByLabelText(/area/i));
|
||||
await user.click(screen.getByRole('option', { name: /front counter/i }));
|
||||
|
||||
await user.click(screen.getByLabelText(/category \(optional\)/i));
|
||||
await user.click(screen.getByRole('option', { name: /drinks/i }));
|
||||
|
||||
await user.click(screen.getByRole('checkbox', { name: /drinks/i }));
|
||||
await user.click(screen.getByRole('checkbox', { name: /snacks/i }));
|
||||
|
||||
await user.click(screen.getByRole('button', { name: /save pick list/i }));
|
||||
|
||||
@@ -23,7 +23,6 @@ export const StartPickListScreen = () => {
|
||||
const [areaId, setAreaId] = useState('');
|
||||
const [notes, setNotes] = useState('');
|
||||
const [selectedCategories, setSelectedCategories] = useState<string[]>([]);
|
||||
const [quickCategoryId, setQuickCategoryId] = useState('');
|
||||
|
||||
const sortedCategories = useMemo(
|
||||
() =>
|
||||
@@ -41,16 +40,6 @@ export const StartPickListScreen = () => {
|
||||
);
|
||||
};
|
||||
|
||||
const handleSelectCategory = (categoryId: string) => {
|
||||
setQuickCategoryId(categoryId);
|
||||
if (!categoryId) return;
|
||||
|
||||
setSelectedCategories((current) =>
|
||||
current.includes(categoryId) ? current : [...current, categoryId],
|
||||
);
|
||||
setQuickCategoryId('');
|
||||
};
|
||||
|
||||
const start = async () => {
|
||||
if (!areaId) return;
|
||||
const pickListId = uuidv4();
|
||||
@@ -120,19 +109,13 @@ export const StartPickListScreen = () => {
|
||||
))}
|
||||
</TextField>
|
||||
<TextField
|
||||
select
|
||||
fullWidth
|
||||
label="Category (optional)"
|
||||
value={quickCategoryId}
|
||||
onChange={(event) => handleSelectCategory(event.target.value as string)}
|
||||
>
|
||||
<MenuItem value="">None</MenuItem>
|
||||
{sortedCategories.map((category) => (
|
||||
<MenuItem key={category.id} value={category.id}>
|
||||
{category.name}
|
||||
</MenuItem>
|
||||
))}
|
||||
</TextField>
|
||||
label="Notes (optional)"
|
||||
value={notes}
|
||||
onChange={(event) => setNotes(event.target.value)}
|
||||
multiline
|
||||
minRows={2}
|
||||
/>
|
||||
<Stack spacing={1}>
|
||||
<Typography variant="subtitle2">Add categories to prefill products</Typography>
|
||||
<FormGroup>
|
||||
@@ -150,13 +133,6 @@ export const StartPickListScreen = () => {
|
||||
))}
|
||||
</FormGroup>
|
||||
</Stack>
|
||||
<TextField
|
||||
label="Notes (optional)"
|
||||
value={notes}
|
||||
onChange={(event) => setNotes(event.target.value)}
|
||||
multiline
|
||||
minRows={2}
|
||||
/>
|
||||
<Button variant="contained" disabled={!areaId} onClick={start}>
|
||||
Save Pick List
|
||||
</Button>
|
||||
|
||||
Reference in New Issue
Block a user