Disabled property for JSONInput (GenericInput)

[alvarolozano]

What does it do?

Describe the technical changes you did.

Why is it needed?

The GenericInput component does not currently pass the disabled attribute to the JSONInput component, so for advanced use cases, it is not possible to achieve a readonly JSON field

How to test it?

Add a new GenericInput to a wiew with type="json" and disabled={true}

Related issue(s)/PR(s)

FIXES #16030

[strapi-cla]

All committers have signed the CLA.

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +5.62 🎉

Comparison is base (87ca2cc) 60.68% compared to head (2a9979a) 66.31%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #16033      +/-   ##
==========================================
+ Coverage   60.68%   66.31%   +5.62%     
==========================================
  Files        1495     1121     -374     
  Lines       36876    22798   -14078     
  Branches     7359     4194    -3165     
==========================================
- Hits        22380    15118    -7262     
+ Misses      12415     6800    -5615     
+ Partials     2081      880    -1201     

Flag	Coverage Δ
back	?
front	66.31% <ø> (ø)
unit_back	?
unit_front	66.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 374 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[markkaylor]

Looks good to me, thanks for the PR!

[markkaylor]

@alvarolozano Everything looks good, I tested and approved. I went ahead and marked your PR as ready for review to merge but you will need to assign the CLA: #16033 (comment) before I can merge.

[alvarolozano]

@alvarolozano Everything looks good, I tested and approved. I went ahead and marked your PR as ready for review to merge but you will need to assign the CLA: #16033 (comment) before I can merge.

Done!