Skip to content

Edit and Delete Review Route Endpoint#59

Merged
Prakhar896 merged 39 commits intomainfrom
lincoln
Jul 19, 2024
Merged

Edit and Delete Review Route Endpoint#59
Prakhar896 merged 39 commits intomainfrom
lincoln

Conversation

@lincoln0623
Copy link
Copy Markdown
Contributor

Overview of Changes

  • Sync-up minor changes required from Jun Han in LoginAccount.jsx
  • Made error handling for every sql transactions
  • Apply delete method in manageReviews.jsx to delete reviews
  • Apply and optimised put method in manageReviews.jsx to update the review if there are any changes ( upon clicking save button on modal )

@lincoln0623 lincoln0623 added the enhancement New feature or request label Jul 14, 2024
@lincoln0623 lincoln0623 requested a review from Prakhar896 July 14, 2024 21:36
@lincoln0623 lincoln0623 self-assigned this Jul 14, 2024
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.

Quite a few changes to ensure proper error handling and consistency.

Comment thread routes/reviews/likeReview.js
Comment thread routes/reviews/likeReview.js Outdated
Comment thread routes/reviews/likeReview.js Outdated
Comment thread routes/reviews/manageReviews.js Outdated
Comment thread routes/reviews/manageReviews.js Outdated
Comment thread routes/reviews/manageReviews.js Outdated
Comment thread routes/reviews/manageReviews.js Outdated
Comment thread routes/reviews/manageReviews.js
Comment thread routes/reviews/manageReviews.js Outdated
Comment thread routes/reviews/manageReviews.js Outdated
@lincoln0623 lincoln0623 requested a review from Prakhar896 July 19, 2024 09:10
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 11f45a9 into main Jul 19, 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