Skip to content

Approx address and MakanHistory filtering bug#101

Merged
Prakhar896 merged 7 commits intomainfrom
joshua
Aug 2, 2024
Merged

Approx address and MakanHistory filtering bug#101
Prakhar896 merged 7 commits intomainfrom
joshua

Conversation

@Sadliquid
Copy link
Copy Markdown
Contributor

What was done in this PR

1.) Implemented adjustements to backend APIs to conform to updated flow of tapping on the host's address and coordinates for all foodListings on frontend

2.) foodListings will update their address and coordinates accordingly upon changing the address of the host

3.) Fixed bug where MakanHistory did not filter the reservations correctly

4.) Updated FoodListing model, renamed coordinates to approxCoordinates (SENSITIVE)

--END OF PR--

@Sadliquid Sadliquid added the enhancement New feature or request label Aug 1, 2024
@Sadliquid Sadliquid requested a review from Prakhar896 August 1, 2024 15:08
Copy link
Copy Markdown
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

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

Well-done, super minor changes needed here and there

Comment thread routes/identity/CreateAccount.js
Comment thread routes/identity/CreateAccount.js Outdated
Comment thread routes/identity/MakanHistory.js Outdated
Comment thread routes/identity/MakanHistory.js
Comment thread routes/identity/myAccount.js Outdated
Comment thread routes/identity/myAccount.js Outdated
Comment thread routes/identity/myAccount.js Outdated
@Sadliquid Sadliquid requested a review from Prakhar896 August 1, 2024 17:18
Copy link
Copy Markdown
Contributor

@Prakhar896 Prakhar896 left a comment

Choose a reason for hiding this comment

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

LGTM

@Prakhar896 Prakhar896 merged commit 3e6e581 into main Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants