unity-addressable-importer icon indicating copy to clipboard operation
unity-addressable-importer copied to clipboard

Multiple AddressableImportSettings support

Open ese9 opened this issue 2 years ago • 2 comments

Update with AddressableImportSettings. Make settings more flexible and readable if you have a lot of rules by spliting to different scriptables.

  • add multiple AddressableImportSettings support
  • add rules enable state toggle
  • add ability to place settings asset in any project folder (not Assets/AddressableAssetsData only)

Screenshot_3 Screenshot_4

Closes #66

ese9 avatar Jan 22 '22 18:01 ese9

Hi @ese9,

Thanks for the contribution. I like the idea. But here's something to be considered:

  1. Instance(s) is being invoked every time an asset is imported. I suppose AssetDatabase.FindAssets("t:... will be a bottleneck for a larger project (especially when performing a bulk reimport).
  2. The ordering of multiple settings should be deterministic. i.e. by introducing a priority field or referencing and ordering multiple settings files into a singleton lister file.

favoyang avatar Feb 06 '22 16:02 favoyang

Please also notice that https://github.com/favoyang/unity-addressable-importer/pull/68 brings some conflicts for the PR.

favoyang avatar Feb 06 '22 16:02 favoyang