[go: up one dir, main page]

Skip to content

Validate boolean attributes on integrations

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

The following discussion from !83206 (merged) should be addressed:

  • @alexkalderimis started a discussion: (+2 comments)

    This row really stands out - if I pass "foobar" I would be suprised if the value was falsey.

    I wonder if we should add validations for all boolean accessor fields? This really stands out as indicating that we are close to having weird behavior on our hands. Also, it is even weirder that if I set x.test_value = 'foobar' then x.test_value.nil?!!

    This is probably worth a follow-up to clarify this odd situation, but it only encodes existing logic.

Edited by 🤖 GitLab Bot 🤖