favoyang / unity-addressable-importer

A rule based addressable asset importer
MIT License
883 stars 124 forks source link

Multiple AddressableImportSettings support #67

Closed ese9 closed 2 years ago

ese9 commented 2 years ago

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

Screenshot_3 Screenshot_4

Closes #66

favoyang commented 2 years ago

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 commented 2 years ago

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