-
Notifications
You must be signed in to change notification settings - Fork 258
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
Feature addon metadata #4167
Feature addon metadata #4167
Conversation
…ture-addon-metadata
…maptool into feature-addon-metadata
Asset asset = | ||
Type.fromMediaType(mediaType) | ||
.getFactory() | ||
.apply(namespace + "/" + METADATA_DIR + path, bytes); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it intentional to add METADATA_DIR
to path
a second time here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you believe me if I said yes? No? yeah I wouldn't either...
I have updated so that the metadata dir doesn't get appended twice to the name of the asset
Identify the Bug or Feature request
resolves #4166
Description of the Change
Any files in the root of the mtlib zip file also get entries in the metadata/ directory
Possible Drawbacks
If someone has a metadata/ directory already that contains the same filename it will be overwritten
Documentation Notes
Files at the top level of the .mtlib zip file will be available lib:///metadata/
Release Notes
This change is![Reviewable](https://camo.githubusercontent.com/23b05f5fb48215c989e92cc44cf6512512d083132bd3daf689867c8d9d386888/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)