-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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: YouTube Video Summarizer #5062
base: master
Are you sure you want to change the base?
Conversation
Hello!I found the following issues in your plugin submission Errors: ❌ Please don't use the word 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][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. |
Thank you for the feedback. I have addressed the styling issues by:
The changes can be seen in commit: |
Changes requested by bot have been made, ready for additional review by human. |
The automated review has not found any issues with your code, ready for manual review. |
callback: () => this.handleSummarizeCommand(), callback: () => this.handleSummarizeCommandWithPrompt() console.log('Loading plugin data...');, console.log('No saved data found, using defaults'); .setName('Gemini API Key'), .setName('Gemini Model'), .setName('Summary Prompt'), .setName('Max Tokens'), .setName('Reset Settings') |
Thank you @Zachatoo for the detailed review! I've addressed all the issues in this commit:
Changes can be reviewed in the commit diff. Let me know if anything needs further adjustment! |
Hello!I found the following issues in your plugin submission Errors: ❌ Your manifest does not have the required 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. |
fixed
|
Hello!I found the following issues in your plugin submission Errors: ❌ Unable to find a release with the tag 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. |
Changes requested by bot have been made, ready for additional review by human. |
Changes requested by Zachatoo have been made, assigning additional human for additional review |
name: 'Summarize YouTube Video',, name: 'Summarize YouTube Video (With Prompt)',, Watch Video, Key Points, Technical Terms api, api return url.includes('youtube.com/') || url.includes('youtu.be/'); |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/mbramani/obsidian-yt-video-summarizer
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
.