-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Add plugin: Private AI #6939
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: Private AI #6939
Conversation
Hello!I found the following issues in your plugin submission Errors: ❌ You did not follow the pull request template. The PR template can be found here 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. |
Hello!I found the following issues in your plugin submission Errors: ❌ Your manifest has the invalid 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. |
Hello!I found the following issues in your plugin submission Errors: ❌ Your manifest has the invalid 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. |
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. |
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1]:The command name should not include the plugin name. [1][2][3][4][5][6]:Using [1][2]: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. Optional[1][2][3][4][5]: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. |
The automated review has not found any issues with your code, ready for manual review. |
'Open Private AI', 'Open Tabs', 'Search Vault', 'Last 7 Days', 'No Context', 'Test Connection', 'Test Connection', 'Test Connection', 'Report a Problem', const chatView = leaf.view as ChatView; containerEl.createEl('h4', { text: 'Private AI' }); systemPromptTextArea.style.width = '100%'; containerEl.createEl('h4', { text: 'Search Settings' }); containerEl.createEl('h4', { text: 'Search Settings' });, containerEl.createEl('h4', { text: 'Support' }); new Setting(containerEl).setName('name here').setHeading(); const file = (leaf.view as unknown as MarkdownView).file;, const activeFile = (activeLeaf.view as unknown as MarkdownView).file; const activeLeaf = this.app.workspace.activeLeaf;
const content = await this.app.vault.read(file);, const content = await this.app.vault.read(file);, const content = await this.app.vault.read(file); await MarkdownRenderer.renderMarkdown(, await MarkdownRenderer.renderMarkdown( this.plugin, this.plugin document.execCommand('copy');, document.execCommand('copy'); "minAppVersion": "0.15.0", LoggingUtility.log('Loading Private AI plugin'); this.messageContainer.addEventListener('keydown', (e) => { |
Thank you for the great feedback @Zachatoo. I've updated the PR and addressed the items you've requested. |
'Open Private AI', 'Open Tabs', 'Search Vault', 'Last 7 Days', 'No Context', 'Test Connection', 'Test Connection', 'Test Connection', 'Report a Problem', Once these are resolved I'm planning on merging. |
Oh derp, you're right I totally missed those. I've fixed them and pushed the requested changes. I double checked with a few LLM's and confirmed that AI should be capitalized since it's an acronym. Happy to change if you disagree. Thank you for your feedback! |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/gabosgab/ObsidianPrivateAI
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
.