Remove string values from boolean data attributes
There are multiple instances in our code where we evaluate the string value of a data attribute to determine it's boolean value. This is not a good practice since it conflicts with https://html.spec.whatwg.org/#boolean-attributes
According to the HTML spec, the presence of a boolean attribute determines whether an attribute is true or false.
It seems like we have a few use cases where we use convertPermissionToBoolean
to check for boolean string value. That is probably a good place to start looking.