-
Notifications
You must be signed in to change notification settings - Fork 111
Create ProjectInterfaces.js #304
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
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Updated the fork im pulling from, it should update the pull right? |
yeah |
arguments: {id: {type: Scratch.ArgumentType.STRING, defaultValue: 'My element'}, dir: {type: Scratch.ArgumentType.NUMBER, defaultValue: 90}} | ||
},{ | ||
opcode: "Scale", | ||
text: "Set scale of ID [id] to wdith: [width]px height: [height]px", |
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.
misspelled width
it didn’t. i’ll try to find out why. |
you didn’t commit your changes. |
Ok, I actually committed the extension code lmao |
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.
its actually good to go for the most part
-Removed console.log calls -Put "lookup" variable in main function -hidden "html attribute" due to xss vulnerabilities
@RedMan13 Fixed |
theeee xss issue is still applicable |
I removed the "Html attribute" block altogether, I searched and discussed with some other members and I couldn't find any other vulnerabilities. (Most of the other ones check to make sure its the right thing) |
my extension 🥳

let me know (preferably on discord) if there are issues.