Skip to content
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

huge readme update and adds a proxy #54

Merged
merged 16 commits into from
Feb 3, 2024

Conversation

skrill-on-discord
Copy link

literally the title, what do you expect to see here?

@penguinify
Copy link
Owner

I would hope you actually at least know the basics of javascript before you contribute, especially if you're making changes to the rendering framework. I also don't think adding proxies is important enough to be explained in the readme. Maybe once I actually get code docs I will move it there in a section.

All the other changes looks good though!

@skrill-on-discord
Copy link
Author

i sorta added it there because proxies die a lot. also i worded that poorly, i meant you dont need to know a lot, obviously the basics

@skrill-on-discord
Copy link
Author

will this be added in v9 perhaps? Perchance if you will.

@penguinify
Copy link
Owner

ill likely remake the readme for v9 so idk atm, we will see once I replace main with the nextgen branch in early february

@skrill-on-discord
Copy link
Author

just fixed a dead proxy

@skrill-on-discord
Copy link
Author

added "- PATCHED" warnings to certain exploits n stuff. just look at the changes thingy

@penguinify penguinify merged commit f77c4c1 into penguinify:main Feb 3, 2024
2 checks passed
@penguinify
Copy link
Owner

yay

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.

2 participants