Skip to content

[SSF-111] food manufacturer application frontend #85

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
amywng wants to merge 32 commits into
base: main
Choose a base branch
from
Open

[SSF-111] food manufacturer application frontend #85

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
ac622e1
fix pantry api body and add manufacturer app backend
amywng Jan 17, 2026
08a983a
make manufacturer attribute optional
amywng Jan 19, 2026
c9741df
review comments
amywng Jan 25, 2026
dd986b5
fix test name
amywng Jan 25, 2026
14afb2a
add not empty validation to pantry/manufacturer app dtos
amywng Jan 25, 2026
b2f494d
fix status everywhere
amywng Jan 25, 2026
cc32e62
prettier
amywng Jan 25, 2026
1283213
review comments
amywng Feb 2, 2026
007ce28
Merge branch 'main' into acw/SSF-107-manufacturer-app-backend
amywng Feb 5, 2026
2e5fce7
Merge main
amywng Feb 5, 2026
b0d4997
add migration
amywng Feb 6, 2026
6f153a9
fix frontend type
amywng Feb 6, 2026
3ced864
prettier
amywng Feb 6, 2026
e123e21
review comments
amywng Feb 9, 2026
ab33cf3
fix api docs
amywng Feb 9, 2026
338dd39
switch post endpoint to /application
amywng Feb 9, 2026
3b5579f
review comments
amywng Feb 11, 2026
bdf2cb3
Merge branch 'main' into acw/SSF-107-manufacturer-app-backend
amywng Feb 11, 2026
0ad71e6
manufacturer app frontend
amywng Jan 19, 2026
7f1886e
add missing separator
amywng Jan 19, 2026
d105bbf
fix api client endpoint
amywng Jan 25, 2026
768ea22
review comments
amywng Jan 26, 2026
2b79d28
add hover blue and fix spacing
amywng Jan 27, 2026
f5947c6
Merge branch 'main' into acw/SSF-111-manufacturer-app-frontend
amywng Feb 12, 2026
dd24fef
fix api client route
amywng Feb 12, 2026
d905cfa
prettier
amywng Feb 12, 2026
c58f290
Merge main
amywng Feb 14, 2026
8ef5556
move manufacturer app to public routes section
amywng Feb 14, 2026
ced4e18
Merge remote-tracking branch 'origin' into acw/SSF-111-manufacturer-a…
amywng Feb 17, 2026
229a0cf
Merge branch 'main' into acw/SSF-111-manufacturer-app-frontend
amywng Feb 17, 2026
fbcfbb3
review comments
amywng Feb 17, 2026
f12cf71
somehow missed prettier
amywng Feb 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions apps/frontend/src/api/apiClient.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ import {
Pantry,
PantryApplicationDto,
CreateMultipleDonationItemsBody,
ManufacturerApplicationDto,
OrderSummary,
UserDto,
OrderDetails,
Expand Down Expand Up @@ -204,6 +205,12 @@ export class ApiClient {
.then((response) => response.data);
}

public async postManufacturer(
data: ManufacturerApplicationDto,
): Promise<AxiosResponse<void>> {
return this.axiosInstance.post(`/api/manufacturers/application`, data);
}

public async getAllOrders(): Promise<OrderSummary[]> {
return this.axiosInstance
.get('/api/orders/')
Expand Down
13 changes: 10 additions & 3 deletions apps/frontend/src/app.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import submitFoodRequestFormModal from '@components/forms/requestFormModal';
import { submitDeliveryConfirmationFormModal } from '@components/forms/deliveryConfirmationModal';
import FormRequests from '@containers/FormRequests';
import PantryApplication from '@containers/pantryApplication';
import PantryApplicationSubmitted from '@containers/pantryApplicationSubmitted';
import ApplicationSubmitted from '@containers/applicationSubmitted';
import { submitPantryApplicationForm } from '@components/forms/pantryApplicationForm';
import ApprovePantries from '@containers/approvePantries';
import VolunteerManagement from '@containers/volunteerManagement';
Expand All @@ -28,6 +28,8 @@ import ForgotPasswordPage from '@containers/forgotPasswordPage';
import ProtectedRoute from '@components/protectedRoute';
import Unauthorized from '@containers/unauthorized';
import { Authenticator } from '@aws-amplify/ui-react';
import FoodManufacturerApplication from '@containers/foodManufacturerApplication';
import { submitManufacturerApplicationForm } from '@components/forms/manufacturerApplicationForm';

Amplify.configure(CognitoAuthConfig);

Expand Down Expand Up @@ -64,8 +66,13 @@ const router = createBrowserRouter([
action: submitPantryApplicationForm,
},
{
path: '/pantry-application/submitted',
element: <PantryApplicationSubmitted />,
path: '/food-manufacturer-application',
element: <FoodManufacturerApplication />,
action: submitManufacturerApplicationForm,
Copy link
Collaborator

@sam-schu sam-schu Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not suggesting a change, just not super familiar with this pattern - what's the benefit of the action function over calling the submit function from the application component?

Copy link
Member Author

@amywng amywng Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think it's better because for separation of concerns, keeping rendering in the component and submission logic in the route action

},
{
path: '/application-submitted',
element: <ApplicationSubmitted />,
},
{
path: '/unauthorized',
Expand Down
Loading