-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat(new features): adds features, read pr for details #9
base: main
Are you sure you want to change the base?
Conversation
src/extension.ts
Outdated
workspace.registerTextDocumentContentProvider("vencord-companion", { | ||
async provideTextDocumentContent(uri) { | ||
console.log(uri) | ||
//FIXME: full uri shows up in title bar | ||
const newLocal = Buffer.from(uri.path.substring(1, uri.path.lastIndexOf(".")), "base64url"); | ||
return newLocal.toString() | ||
}, | ||
}), |
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.
this seems like a really bad way to create a diff between two strings, but I tried for like a few hours and I couldn't figure out any other way
update esbuild update build system add webview to normal build system update scripts compile => build
I think all the features are done, but this is still nowhere near ready
review is appreciated from whoever offers it |
Needs review
needs to be merged with the matching pr in the vencord repo
if you have any ideas for features, ping me on discord
current planned features
https://github.com/user-attachments/assets/7d4ab157-0751-4a59-8e0e-1a3607a3247d