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

Updated FunctionUtil to handle lib:// assets via AssetResolver #4629

Merged
merged 3 commits into from
Jan 12, 2024

Conversation

ColdAnkles
Copy link
Contributor

@ColdAnkles ColdAnkles commented Jan 11, 2024

Identify the Bug or Feature request

fixes #4626
resolves #4620 in a smarter fashion

Description of the Change

Updates FunctionUtil.getAssetKeyFromString() to handle both asset:// and lib:// asset URLs
Propagates that usage to both createMap, and input functions.

Possible Drawbacks

None

Documentation Notes

FunctionUtil now depends on AssetResolver imports
InputFunction now depends on FunctionUtil

Release Notes

Fixed several inconsistent inabilities to use lib:// assets


This change is Reviewable

Updated MapFunctions CreateMap to use this.
Updated InputFunction to use this.
@ColdAnkles ColdAnkles changed the title Updated createMap to handle lib:// assets via AssetResolver Updated FunctionUtil to handle lib:// assets via AssetResolver Jan 12, 2024
@ColdAnkles
Copy link
Contributor Author

If we can find any similar uses of asset:// only I can update for those as well.

@bubblobill
Copy link
Collaborator

sooo... if I have an image: token, what happens?

@ColdAnkles
Copy link
Contributor Author

sooo... if I have an image: token, what happens?

Was you'd get a null, but now it should work, though I suspect there's a better way to implement.

Probably should discourage using Image token names directly in this way, since afaik token names aren't unique.

@cwisniew cwisniew added the bug label Jan 12, 2024
@cwisniew cwisniew added this pull request to the merge queue Jan 12, 2024
Merged via the queue into RPTools:develop with commit de08ce2 Jan 12, 2024
4 checks passed
@ColdAnkles ColdAnkles deleted the createMap-lib-asset branch January 12, 2024 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: createMap does not accept add-on URIs [Feature]: Input() Icon to Accept lib:// image assets
4 participants