-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Add plugin: BibDesk Integration #4569
Conversation
Hello!I found the following issues in your plugin submission Errors: ❌ Plugin name mismatch, the name 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. |
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1]:Using [1][2][3][4][5][6][7][8][9]:You should avoid assigning styles via JavaScript or in HTML and instead move all these styles into CSS so that they are more easily adaptable by themes and snippets. Do NOT open a new PR for re-validation. |
Changes requested by bot have been made, assigning human for additional review. |
Your release constructor(app:App,manifest:PluginManifest) { // Path to plugins folder function max(a:number,b:number): number { |
Thanks! I did all changes. PS: Do you get notificiations when I post a message like this, or I need to tag you to inform you when changes are made? |
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.
Made requested changes to bibdesk-integration
. New version 1.0.2 available.
Dear Jothei, what does the flag "Installation not recommended" mean? Is the plugin rejected or some changes are needed? |
Ok. I think I understood why the flag |
Sorry for the delay. # Obsidian BibDesk Integration "description": "Allows using BibTex citations and opening attached PDFs from a BibDesk library.", |
Thanks! Yes, I changed the header in the README.md and updated the description. The new changes are included in release 1.0.3 |
Hi there, as this PR has not seen any activity in the last 30 days, it will be closed in 15 days unless there are any updates. |
@joethei I know you have a lot of work and probably some backlog. Is there anything more has to be done on this plugin? It was nicely crafted, and I am sure it will be helpful for people to manage their scientific library. It is many months since it was submitted. Thanks for your consideration. |
@joethei Is there anything I should do on my side or simply wait? Perhaps you are simply busy with other projects? A short answer would be greatly appreciated. |
You have not updated the description in the manifest. |
@joethei Thanks! I had changed Now the new release 1.0.4 includes the new description. |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/alberti42/obsidian-bibdesk-integration
Release Checklist
main.js
manifest.json
styles.css
(optional)v
)id
in mymanifest.json
matches theid
in thecommunity-plugins.json
file.I have given proper attribution to these other projects in my
README.md
.