Skip to content

Conversation

@Juwang110
Copy link

Removed unused status fields for donation_items and allocations

Copy link
Member

@amywng amywng left a comment

Choose a reason for hiding this comment

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

can you fix the frontend types too and merge main

Copy link

@dburkhart07 dburkhart07 left a comment

Choose a reason for hiding this comment

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

Aside from Amy's comments, lgtm!!

@Juwang110 Juwang110 requested a review from amywng December 7, 2025 15:39
@Juwang110 Juwang110 requested a review from amywng December 7, 2025 17:17
Copy link
Member

@amywng amywng left a comment

Choose a reason for hiding this comment

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

lgtm

@Juwang110 Juwang110 merged commit 76319ab into main Dec 9, 2025
3 of 4 checks passed
@Juwang110 Juwang110 deleted the jw/removing-unused-statuses branch December 9, 2025 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants