-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: add mimetypes to nextcloud core #78
base: main
Are you sure you want to change the base?
Conversation
Thanks for the PR See registerForExistingFiles and registerForNewFiles Let us know if you have any other questions |
I saw those functions, they are adding the mime types to the correct files, which are However, on my system, the I'm trying to understand how my |
Sorry for the delay, I wanted to dig deeper but I'm still busy |
Are these images important? What are they used for? |
They are the thumbnail images used to represent items of the given MIME type |
@tnyeanderson Are you blocked by us? I hope I answered all your questions Mindmap added their mimetypes in this PR and commit, I think nextcloud has a standard process to add a new type but I forgot where I read about that: |
At this point, i suspect that once the stuff is added upstream and this is merged, things will be fine again (maybe this is a quirk of my system which is many years old and has been through many updates--who knows). I'll work on finalizing the upstream PR so this can be merged. Sorry for my delay |
Fixes: #26
I've opened a PR for nextcloud server which adds the drawio mimetypes upstream to prevent code signing errors.
The above issue was locked to collaborators due to excessive "+1" posts, but this stopped the implementation conversation as well.
This PR prevents the installer from copying the SVG mimetype icons.
TODO
Where in the code does the
core/js/mimetypelist.js
file get manipulated? I'm not familiar with the nextcloud interfaces, but that will need to be removed as well.