box / box-content-preview

JavaScript library for rendering files stored on Box
https://developer.box.com/docs/box-content-preview
Other
106 stars 113 forks source link

feat(aci): use feature check for aci #1508

Closed arturfrombox closed 1 year ago

arturfrombox commented 1 year ago

This pr introduces changes to cover ACI codebase with feature checking

arturfrombox commented 1 year ago

What's benefit of these changes if we already have a configuration object for these options? We appear to be increasing complexity at every layer.

It was one of PMAI's and intention behind this change was to stick to "feature checking" mechanism implemented for Preview and to hide rest of ACI codebase under the feature flip.

arturfrombox commented 1 year ago

@ahorowitz123 @jstoffan thanks for your comments. I updated this PR