Skip to content

South dox patch 1#756

Closed
yusufomer08 wants to merge 3 commits into
Vendicated:mainfrom
yusufomer08:SouthDox-patch-1
Closed

South dox patch 1#756
yusufomer08 wants to merge 3 commits into
Vendicated:mainfrom
yusufomer08:SouthDox-patch-1

Conversation

@yusufomer08
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Contributor

@144reasons 144reasons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, your pr should have at least the plugin name/what you are adding, like so: feat(Plugin): Typso

You should also add a short description if possible. Other than my small suggestions, everything looks relatively fine, but I am sure there are some small improvements available but cant see any rn. Thanks for the pr!

Comment thread src/plugins/Typso.ts
export default definePlugin({
name: "Typso",
description: "Mke yoyr fruends thibk you'te typinf woth yoyr toes!",
version: "1.0.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Version doesn't exist in plugin definiton

Comment thread src/plugins/Typso.ts

addPreSendListener((_, msg) => {
var newMsg = "";
let guh = false;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use something more descriptive than guh, it took me a whole minute to decipher what guh was meant to do, and still dont fully understand. Use something like isLink

@Vendicated
Copy link
Copy Markdown
Owner

what

@Vendicated Vendicated closed this Apr 2, 2023
@144reasons
Copy link
Copy Markdown
Contributor

didnt see this was a literal copypaste lmfao

EtorixDev pushed a commit to EtorixDev/Vencord that referenced this pull request Feb 11, 2026
* FileUpload: add 0x0 and catbox

* FileUpload: add litterbox
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants