Skip to content

Update specs for integrations API to check all modifiable fields

Markus Koller requested to merge 348461-integration-api-put-specs into master

What does this MR do and why?

Previously only the event booleans were verified, but not any other attributes that were sent in the PUT request.

This is a step towards fixing inconsistencies in the Integrations API, some attributes are currently not supported and skipped in the specs.

This change also exposed a bug in the getters defined by data_field, where we'd incorrectly fallback to properties for falsy values. This should only be done for explicit nil values.

Issue: #348461 (closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Markus Koller

Merge request reports