-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Add plugin: Movie Companion #5713
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
Conversation
Adding Movie Companion Plugin.
Hello!I found the following issues in your plugin submission Errors: ❌ Your latest Release is missing the 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. |
file uploaded |
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1]:Please remove the [1][2][3][4][5][6][7][8][9][10][11][12]:You should change all instances of Optional[1][2]:Use [1]:Casting to Do NOT open a new PR for re-validation. |
Changes requested by bot have been made, ready for additional review by human. |
Hello!I found the following issues in your plugin submission Errors: ❌ The newly added entry is not at the end, or you are submitting on someone else's behalf. The last plugin in the list is: 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. |
Ok, I've made all of the required changes. What about setting the language in the config? This is needed because people might have Obsidian in one language (e.g. en) but want to collect their movies in another (e.g. it). |
Hi @Zachatoo, is there anything else I can do? |
@ats4e Can you double check that you made all the requested changes? The bot looks through each link I made to your code and checks to see if you've made a change to that code. If you believe you have made all the requested changes then I can check it on my end. |
Hi @Zachatoo, I have checked all the controls I could. The only one I’m unable to verify is Android, as I don’t own any Android device. |
The bot checks each comment I make and compares it against your code to see if it changed. These are the comments that I made that did not change, so the bot did not move this forward. new Setting(containerEl).setName("TV Shows").setHeading();, .setName("New TV Shows file location"), .setDesc("New TV Shows notes will be placed here."), .setName("TV Show template file") function replaceIllegalCharsFromPath(text: string) { const manualCollectionDesc = document.createDocumentFragment(); console.log( Here are some additional comments: const manualCollectionDesc = document.createDocumentFragment(); |
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. |
Hi there, to keep things tidy, we're closing PRs after one and a half months of inactivity. |
Adding Movie Companion Plugin.
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/ats4e/obsidian-movie-companion
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
.