GRESB / api-docs

http://gresb.github.io/api-docs/
4 stars 0 forks source link

verify if all request examples still work in the api-docs #88

Closed joeyhorst closed 2 years ago

joeyhorst commented 2 years ago

as the title mentioned, we should run though the examples and make sure they are all valid

naincygoyal commented 2 years ago

@joeyhorst I will test following request

  1. ASSET:

"delete": [ { "gresb_asset_id": 833336 }, { "gresb_asset_id": 833337 } ] }

**Response Screenshot** ![image](https://user-images.githubusercontent.com/97026424/155539218-71b623c2-7d28-4a71-ac84-977a7ff0c3ea.png)
Sample Request Body { "always_update": [ { "gresb_asset_id": 833337, "annual_data": [ { "year": 2021, "property_type_code": "OCHI" } ] },{ "gresb_asset_id": 833338, "annual_data": [ { "year": 2021, "property_type_code": "OCHI" } ] } ] }
Sample Response Body ![image](https://user-images.githubusercontent.com/97026424/155540560-6a8d2e5f-47d9-432c-864d-c54db0448de2.png)

However , these are the assets which I created new.


  1. ENTITY
  1. ENERGY RATINGS
  1. CERTIFICATION
  1. USER
    • [x] GET Request to retrieve all the authenticated users - No Modification require in API Docs.
joeyhorst commented 2 years ago

@naincygoyal believe we need to cover all code blobs on the right hand side of the api-docs - such as http://gresb.github.io/api-docs/#users, etc. This also includes simple GET requests.

naincygoyal commented 2 years ago

Yes @joeyhorst even Certification and Energy need to add ...I will first finish asset then will pick user, energy , certification etc and will update in the same list.

naincygoyal commented 2 years ago

@joeyhorst https://github.com/GRESB/api-docs/issues/88#issuecomment-1049779332 in this comment I mentioned PATCH and DELETE in batch is not working , could you please once and confirm why its not working ?

joeyhorst commented 2 years ago

@naincygoyal The batch operation contains objects like "always_update" and "delete" to perform PATCH and DELETE methods.

As such you will need to do this within the batch operation and not as a method.

The HTTPS method to use is POST.

joeyhorst commented 2 years ago

closing this as it is already in master but still needs to be published