beckn / transactional-ai-bap

MIT License
7 stars 4 forks source link

MIT Catalog feedback - Review with Mayur on April 10 #77

Closed meenakshi2468 closed 7 months ago

meenakshi2468 commented 7 months ago

Description

For the upcoming MIT demo on April 11, 80+ Catalogs were created for three domains: Hotel, Retail and EV. These were reviewed with Mayur on April 10, 2024. As the AI chatbot was showing error due to 'provider uri' added in the catalogs, Mayur suggested to remove 'provider URI' from all catalogs.

Feedbacks received

1. Remove 'provider URI' from catalogs
2. Bank details to be updated with American context 
3. Logo URL - Mayur to confirm if its to be removed from catalogs
4. How search is happening - based on Fulfillment location/item location - Vishwa to check with Abhishek
5. Removed 'Size Type' from Media - to resolve protocol server issue (PS not able to process that)
7. 'Tags' and 'Categories' - Mayur to confirm if this need to be fixed
8. Validate Provider location ID = Item-fulfillment  location id - match with  Provider location ID

Team to work on these issues

meenakshi2468 commented 7 months ago

As discussed and confirmed with Mayur, he will be handling the 'provider URI' processing in the AI bot code. Hence, this is not required and closed.

meenakshi2468 commented 7 months ago
1. Remove 'provider URI' from catalogs - Done
2. Bank details to be updated with American context  - Done
3. Logo URL - Mayur to confirm if its to be removed from catalogs - pending
4. How search is happening - based on Fulfillment location/item location - Vishwa checked with Abhishek, search is based on item fulfillment
5. Removed 'Size Type' from Media - to resolve protocol server issue (PS not able to process that) - Vishwa resolved this
7. 'Tags' and 'Categories' - Mayur to confirm if this need to be fixed
8. Validate Provider location ID = Item-fulfillment  location id - match with  Provider location ID - Done