Skip to content

Conversation

@bradley-p
Copy link
Contributor

This pull request enhances the flexibility and usability of the Procore API access layer by adding an option to return raw requests.Response objects from all HTTP request methods in the Base class. It also updates the documentation for these methods, improves parameter handling, and exposes the Base API access class through the main Procore class.

API response flexibility and improvements:

  • Added a return_request_obj boolean parameter to all HTTP methods (get_request, post_request, patch_request, delete_request) in Base, allowing callers to receive either the parsed response (default) or the raw requests.Response object. This change is reflected in both the method signatures and their docstrings. [1] [2] [3] [4] [5] [6] [7] [8] [9]
  • Improved docstrings for all HTTP methods in Base to clearly document the new parameter and return types. [1] [2] [3] [4]

Codebase and interface enhancements:

  • Exposed the Base API access class as base_api on the main Procore class, making it accessible for generic API calls. [1] [2]
  • Made minor formatting and comment improvements for clarity and consistency in the Base class. [1] [2] [3] [4]

These changes make the API client more versatile for advanced use cases, such as custom response handling, debugging, and integration with other tools without modifying any existing usage or functionality.

…eter. This allows functionality for endpoints that do not return a json body
@HagenFritz HagenFritz self-assigned this Dec 22, 2025
@HagenFritz HagenFritz added the enhancement New feature or request label Dec 22, 2025
@HagenFritz
Copy link
Member

Love it! I find myself running into similar issues where I don't get back all the variables I need. Thanks for the contribution!

@HagenFritz HagenFritz merged commit d93971a into rogers-obrien-rad:main Dec 22, 2025
1 check failed
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