Skip to content

Review Email, Host Availability in Reviews Page and Reviews Page Image Filtering#107

Merged
Prakhar896 merged 20 commits intomainfrom
lincoln
Aug 7, 2024
Merged

Review Email, Host Availability in Reviews Page and Reviews Page Image Filtering#107
Prakhar896 merged 20 commits intomainfrom
lincoln

Conversation

@lincoln0623
Copy link
Copy Markdown
Contributor

Overview of Changes

  • Added chargeableCancelActive, approxAddress & reviewsCount fetching fields in /accountInfo route endpoint.
  • Images tabs in Reviews page will now filter out to display reviews with image(s) only, descending order is still there which means it will still sort according to the amount of image posted in the review, the more will display first.
  • Reviews page will have a 1 minute cooldown upon each review submission for each guests.
  • manageGuests.js/togglePaidAndPresent will update host and guest mealsMatched column when this endpoint is called and triggered, +1 if marked paid and present, -1 vice versa.
  • Now, an review received email will be sent to the host email address if there is a review submitted by a guest to the host.

@lincoln0623 lincoln0623 added the enhancement New feature or request label Aug 2, 2024
@lincoln0623 lincoln0623 requested a review from Prakhar896 August 2, 2024 15:09
@lincoln0623 lincoln0623 self-assigned this Aug 2, 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.

Well-done PR, minor performance changes needed.

Comment thread routes/cdn/coreData.js Outdated
Comment thread routes/orders/manageGuests.js Outdated
Comment thread views/emails/ReviewReceived.html Outdated
@lincoln0623 lincoln0623 requested a review from Prakhar896 August 6, 2024 16:15
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 ab8eea9 into main Aug 7, 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