BrayanBot icon indicating copy to clipboard operation
BrayanBot copied to clipboard

Changes to eval.js

Open Sanchit-codes opened this issue 2 years ago • 5 comments

Updated user interface a bit and fixed an issue when the output goes more than 1024 char then field limit is surpassed and an error is created so to avoid it, the output is posted to hastebin and alternatives.

Sanchit-codes avatar Jun 04 '22 05:06 Sanchit-codes

Removed node-superfetch, now using Utils.paste(). https://i.sanchit.codes/MidnightblueBison-Discord.png

Sanchit-codes avatar Jun 04 '22 07:06 Sanchit-codes

Looks good to be merged. As a sidenote, I would appreciate if pastelink corresponded to an option in config (e.g. pastelink: https://paste.brayanbot.dev) instead of being hardcoded, but that can be done later in collaboration with other paste related improvements

NotAShelf avatar Jun 05 '22 08:06 NotAShelf

Good Suggestion @NotAShelf I'll add it today!

Sanchit-codes avatar Jun 06 '22 03:06 Sanchit-codes

Might break please check it once!

Sanchit-codes avatar Jun 06 '22 16:06 Sanchit-codes

@DakshNamdev @xcraftman52 I am re-requesting your reviews as changes to both this PR and the dev branch have been made. Can be merged if both of your reviews are positive.

NotAShelf avatar Jul 16 '22 10:07 NotAShelf