-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Image Toolkit Plugin #7597
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
Image Toolkit Plugin #7597
Conversation
I would suggest that it is better to edit this change so that the existing entry remains in place, so that it is clear that the only differences are the ownership. That is how past changes in ownership have been made. |
The error checking may grumble that you are not editing the last entry but you can ignore that, if it happens. |
Or you put is another way, you aren’t submitting a new plugin, you are taking over an existing one…. |
Yeah, I was looking for the docs on how to do that but I couldn't find anything. So, do I just edit the line where the previous maintainer's name was and put my own? |
Hello!I found the following issues in your plugin submission Errors: ❌ Plugin ID mismatch, the ID in this PR ( This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it. |
Yes, and make sure that that is the only difference in the file. |
Hm.... Changes can be cleanly merged but the validation failed? I messed up with which version of the community-plugins.json I copied previously (hence the rogue plugin at the bottom - still a PR I am waiting for review) but I have fixed that and yet still validation failed. I guess I am still waiting for more checks? |
The diff looks good. The Obsidian team may have a suggestion about the wording of the author field here and in your fork - like how to make it clear that you are maintaining it but to give credit to the original author too. I don’t know what their policy is. But what you have currently looks like it’s the minimum viable diff. |
Unfortunately your fork of the repo has not pulled in all the existing issues and pull requests. I don’t know if it is possible to do that, but it would really help if it were possible. I would suggest asking on #plugin-dev for advice on what you need to do to take over an existing plugin. |
I suspect, from a Google search, that retaining issues and pull requests is only possible if ownership of the previous repo is transferred. |
It appears there is a way to download all existing issues and move them create new ones on my repo - So I will do that and link between them at the old repo. |
Sorry, but given your track record with maintaining Obsidian related things we are not comfortable giving you ownership of this plugin. Sorry about that! |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/DudeThatsErin/obsidian-image-toolkit
Release Checklist
main.js
manifest.json
styles.css
(optional)v
)id
in mymanifest.json
matches theid
in thecommunity-plugins.json
file.README.md
.