search

beckn / transactional-ai-bap

MIT License
6 stars 4 forks source link

Postman collection for Hotel booking #23

Closed vishi24 closed 6 months ago

vishi24 commented 6 months ago

Description

Create postman collection for the hotel booking use case details are as below base_url - https://mit-ps-bap-client.becknprotocol.io bap_id - mit-ps-bap.becknprotocol.io bap_uri - https://mit-ps-bap.becknprotocol.io

Further details for registry , gateway , bpp are here - https://docs.google.com/spreadsheets/d/1X6RPNJH2eQrpjvHtZte9Jw6Zy0fIj7MNOcdlpr5iphE/edit#gid=2086670707

It should have, search, select, init confirm, cancel, and status endpoints.

Postman collection should be published under https://github.com/beckn/beckn-action-bot/artefacts/postman_collection

Link to catalogs sheet: https://fide-official.atlassian.net/wiki/spaces/DP/pages/293208066/Hotel+Catalog

The collection must use protocol server client of BAP as the base URL.

Goals

Expected Outcome

Postman collection for hotel use case that points to BAP protocol server.

Acceptance Criteria

faizmagic commented 6 months ago

@zoyaashik< please provide an update

faizmagic commented 6 months ago

Current update

  1. Need to upload the catalogs
  2. Following information are missing in the catalog sheet, but has been added to the bpp instance a. Fulfillments
    1. item fulfilment
    2. order fulfilment
    3. provider fulfilment b. Agent information c. cancellation terms d. payment information e. data for the support end points

@pri233, to assign this to another resource to get the catalogs updated in the confluence sheet https://fide-official.atlassian.net/wiki/spaces/DP/pages/292356099/Tourism+Ticket+Booking+Catalog

zoyaashik commented 6 months ago

The hotel domain in postman collection is yielding empty response for search endpoint. Debugging and troubleshooting the logs to find root cause

zoyaashik commented 6 months ago

The root cause was provider image url in DB was too long. Hence shortenined url is updated. Created collection for search,select, init endpoints

pri233 commented 6 months ago

@em-abee Plz get this closed asap

mayurvir commented 6 months ago

Reviewed and merged, any issues discovered after this will be treated as separate bugs.