reapit / foundations

Foundations platform mono repo
57 stars 21 forks source link

Cannot find a property from a customer via the API #7203

Closed enricsangra closed 2 years ago

enricsangra commented 2 years ago

Describe the bug When trying to get a property via ID, the API is returning a 404. According to the customer, this is in their Reapit platform.

To Reproduce Steps to reproduce the behaviour eg:

  1. Call the endpoint: /properties/LMR160023 with the client id NOC.
  2. 404 is returned

Expected behaviour The API should be able to return the data instead of returning a 404.

Screenshots We have screenshots from the customer but they all contain a lot of personal information.

Device (please complete the following information for web issues):

Additional context Ticket submitted by Goodlord regarding the API integration.

This happened in the past with a different company and it was related to some bad data, in case that helps.

github-actions[bot] commented 2 years ago

Thank you for taking the time to report a bug. We prioritise bugs depending on the severity and implications, so please ensure that you have provided as much information as possible. If you haven’t already, it really helps us to investigate the bug you have reported if you provide ‘Steps to Replicate’ and any associated screenshots. Please ensure any personal information from the production database is obscured when submitting screenshots. This issue will be reviewed in our weekly refinement sessions and assigned to a specific project board. We may also update the ticket to request additional information, if required. For more information on our processes, please click here

plittlewood-rpt commented 2 years ago

@AshDeeming this is due to bad data in a few records in the customer's system. Please can you liaise with service and ask them to clean up records with a bad pDepReq value.

AshDeeming commented 2 years ago

RSD-188995

plittlewood-rpt commented 2 years ago

@enricsangra this is now resolved and was caused by bad data in the system. The Platform will omit these records rather than throwing an error for a better developer experience. Thanks