-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Add plugin: Note to RED #5689
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: Note to RED #5689
Conversation
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][10][11][12][13][14][15][16][17][18][19][20][21][22][23][24][25][26][27][28][29][30] and more :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. [1][2][3][4][5][6][7][8][9][10][11][12]:Using [1]:Do not detach leaves with your custom view in Do NOT open a new PR for re-validation. |
I have pushed all the necessary changes to the repo |
I have pushed all the necessary changes to the repo again. |
Changes requested by bot have been made, ready for additional review by human. |
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1][2][3][4][5]: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, ready for additional review by human. |
private updateTimer: NodeJS.Timeout | null = null; |
Thank you very much for your review. I have fixed this issue and submitted it to the main branch. I have also repackaged the releases. Could you please review it again? private updateTimer: number | null = null;
|
The automated review has not found any issues with your code, ready for manual review. |
I am submitting a new Community Plugin
Repo URL
Link to my plugin:https://github.com/Yeban8090/note-to-red
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
.