Skip to content

Extended Listing Host View (Backend)#87

Merged
Prakhar896 merged 23 commits intomainfrom
lincoln
Jul 31, 2024
Merged

Extended Listing Host View (Backend)#87
Prakhar896 merged 23 commits intomainfrom
lincoln

Conversation

@lincoln0623
Copy link
Copy Markdown
Contributor

Overview of Changes

  • /getHostPaymentQR return you the host payment QR code image regardless what type of user you are (guest, host or admin), if you provide the correct food listing listingID, the specific host hostID and your currently login jwt token for authorization. jwt token will used to verify current log in user as an alternative if checkUser authorisation failed
  • /manageGuests where handles upload payment QR code image (/uploadPaymentQR) and toggling of guest paid and present status (/togglePaidAndPresent)

Sensitive Changes!

  • Removed /getReservation route endpoint

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

Minor changes needed

Comment thread routes/orders/manageGuests.js Outdated
Comment thread routes/orders/manageGuests.js Outdated
Comment thread routes/orders/manageGuests.js Outdated
Comment thread routes/orders/manageGuests.js Outdated
Comment thread routes/orders/manageGuests.js Outdated
Comment thread routes/orders/manageGuests.js
@lincoln0623 lincoln0623 requested a review from Prakhar896 July 31, 2024 06:52
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 1f95e68 into main Jul 31, 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