search

silverstripe / silverstripe-asset-admin

Silverstripe assets gallery for asset management
BSD 3-Clause "New" or "Revised" License
20 stars 79 forks source link

ENH Use archive text when file archiving is enabled #1378

Closed emteknetnz closed 1 year ago

emteknetnz commented 1 year ago

Issue https://github.com/silverstripe/silverstripe-versioned-admin/issues/300

AssetAdmin::getClientConfig() is an array of PHP values that are made available in JS

This pull-request is NOT updating jest tests, instead they will be updated in https://github.com/silverstripe/silverstripe-asset-admin/pull/1379 instead

I'm doing this because I want to test https://github.com/silverstripe/gha-merge-up/pull/6 going cross major, though jest tests will definately cause merge-conflicts due to the jest tests being rewritten in 2 https://github.com/silverstripe/silverstripe-asset-admin/pull/1355

After merging - manually run the merge-up action on asset-admin using the default branch - https://github.com/silverstripe/silverstripe-asset-admin/actions/workflows/merge-up.yml