Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addressables Reimport API #21

Closed
Mefodei opened this issue Apr 23, 2020 · 7 comments
Closed

Addressables Reimport API #21

Mefodei opened this issue Apr 23, 2020 · 7 comments
Labels

Comments

@Mefodei
Copy link
Contributor

Mefodei commented Apr 23, 2020

Hello there.
I try to call package API "Reimport Addressable Group (Check Folders)" on Project Build Execution step. But all general static functions is private. Could you allow use Common method of your package as public API?

@favoyang
Copy link
Owner

Hi @Mefodei,

I assume you mean to make "Assets/AddressablesImporter: Check Folder(s)" publicly available? That method only works for the current selection. Is that suitable for your case?

@Mefodei
Copy link
Contributor Author

Mefodei commented Apr 24, 2020

Hi
I'm looking for API method, that takes project folder as parameters and apply reimport action. For example it can be all rules folders from asset Addressables Import Settings Rules.

favoyang added a commit that referenced this issue Apr 24, 2020
@favoyang
Copy link
Owner

That makes sense to me.

I have prepared a branch with a reimport API, which requires an array of folder asset paths. An asset path is a string starts with "Assets". E.g. to process the whole Assets folder (this can be slow depends on the number of assets).

AddressableImporter.FolderImporter.ReimportFolders(new string[] { "Assets" });

To test it, switch your lib to this Git URL.

{
    "dependencies": {
        "com.littlebigfun.addressable-importer": "https://github.com/favoyang/unity-addressable-importer.git#reimport-api"
    }
}

Let me know if this works for you, then I will merge back to the master branch.

@Mefodei
Copy link
Contributor Author

Mefodei commented Apr 27, 2020

Thx, i will try

@Mefodei
Copy link
Contributor Author

Mefodei commented May 7, 2020

Works fine :)

@Mefodei Mefodei closed this as completed May 7, 2020
favoyang added a commit that referenced this issue May 7, 2020
github-actions bot pushed a commit that referenced this issue May 7, 2020
# [0.7.0](v0.6.0...v0.7.0) (2020-05-07)

### Features

* re-import API (close: [#21](#21)) ([8b00747](8b00747))
@github-actions
Copy link

github-actions bot commented May 7, 2020

🎉 This issue has been resolved in version 0.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@favoyang
Copy link
Owner

favoyang commented May 7, 2020

@Mefodei, merged to master and released v0.7.0. The reimport-api branch is deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants