when you click it, server guides completely turns into home, and when you leave the home it turns back to server guide
#👾-core-development
limber skiff
inland fable
wgat if theres no guide
limber skiff
then it shows the home normally lol
the guide overrides the home
this is what it shows if there is no guide
charred monolithBOT
lime stone
how impossible is it to show both
limber skiff
not impossible but bad
Server Guide and Server Home share the same route
this honestly is not a bad solution and was pretty straight forward to make
shy veldt
its so over guys
austere talon
shy veldt
<:skillissue:1165700808348811425>
the FUCK you mean skill issue
and they are doing something
be scared or something
this exp causes it
thing is it still does that even if you dont have the exp
lime stone
Server Guide and Server Home share the same route
don't some other things too
i think it's weirder for ux to merge
austere talon
theres a switch button
lime stone
yeah, that's weird
but i guess fixing the route would be hard
it's just it still suffers from this issue xd
rugged spire
but i guess fixing the route would be hard
nop
i have accidentally ran into the "@home" lying around
lime stone
probably should fix weeks of messages being nuked often happening when interacting with server home xD
rugged spire
reddit app behaviour
tropic wagon
me
tropic wagon
hi anu
why the new user thingy
did you die
beast
send me the link i am one lazy motherfucker
rugged spire
Use `=> ({ ... })`, not `=> { return { ... } }`
Oh ok
FUUUUUUUUCK (Where did my big quick react menu go?)
discord really did decide "good morning, fuck you, rewrite your plugins because we pushed a change to canary" 
prime dew
discord really did decide "good morning, fuck you, rewrite your plugins because ...
discord devs trying not to break stuff challenge (impossible)
how much do you have to rewrite tho?
rugged spire
how much do you have to rewrite tho?
I already rewrote most of a plugin I made when I was starting to make plugins
tropic wagon
message logger moment
austere talon
stop contributing to the spam just ignore retards
tropic wagon
i can see history
rugged spire
I really need to fix that better quick react plugin I made
and quickly
why did it even break 
i may be a bit stupid
pretend i said nothing
?????
limber skiff
fully restart discord
rugged spire
FUUUUUUUUCK (Where did my big quick react menu go?)
in case anyone is still wondering, yes I was panicking about it being broken, but no warning in console
i wasn't even loading the plugin i was on a branch without it
austere talon
do you even know why that error occurs
because vencord's settings ipc was changed from sending strings to sending proper objects
ctrl r only reloads renderer
so you had old ipc with new renderer
rugged spire
OHHHHHH
is this related to the new settings api changes
and that I haven't merged dev into half my branches
brazen bone
and that I haven't merged dev into half my branches
Criminal
crude hearth
horror polls are real
austere talon
its experiment
rugged spire
the old one wasn't bad
You are right, it wasn't bad
It was just very repetitive
lime stone
it's a bit ew doing \\ in regex
shy veldt
@austere talon 
austere talon
he already had a selector for it so why not just use css lmao
rugged spire
mutationobserver with querySelector
vencord potential contributors making the most unoptimised code
LMFAOO
austere talon
<@343383572805058560> <:WHAT:1082407334510350368>
shouldnt matter
brazen bone
Well, renaming things from default to sensible names is a good change imo
shy veldt
yeah
austere talon
Well, renaming things from `default` to sensible names is a good change imo
whar
they dont name it default xd
its just a default export
brazen bone
Well yeah
austere talon
const HTTP = { ... };
// before
export default HTTP;
// now
export { HTTP };
austere talon
personally im hater of default exports i always use named exports but both are fine
shy veldt
you dont get intellisense in console
naw i'd get
austere talon
nop
charred monolithBOT
shy veldt
L
limber skiff
can I have some basic feedback on new webpack api, webpack.tsx is mainly the only file needed to look a lot
charred monolithBOT
limber skiff
super hacky stuff
median rapids
ur a genius
glass jasper
damn
charred monolithBOT
austere talon
@spark pivot this reverts your change to CheckedTextInput because its just wrong
it triggers onChange events when it didnt actually change
charred monolithBOT
austere talon
if you actually need that change for it to function properly, then you're doing something else wrong and should fix that instead!
spark pivot
that pr was from a while ago so i prob did something wrong
charred monolithBOT
dapper crest
where are we supposed to host video files for vencord.dev/plugins now that discord is dumb and stupid
spark pivot
github
dapper crest
ah
spark pivot
paste the image into the readme images thread and get the url
write a meaningful alt text () or leave it blank if the image is not important for understanding functionality (for example it just demonstrates what's already...
dapper crest
ah
austere talon
that's actually gonna go soon replaced with https://github.com/Vencord/plugin-assets
but it's not ready yet
spark pivot
if you actually need that change for it to function properly, then you're doing ...
i don't think i can modify the patch helper code to do this- the way CheckedTextInput is made right now doesn't allow you to change value to update it. I changed it to this and it worked:
-export function CheckedTextInput({ value: initialValue, onChange, validate }: TextInputProps) {
+export function CheckedTextInput({ value, onChange, validate }: TextInputProps) {
- const [value, setValue] = React.useState(initialValue);
const [error, setError] = React.useState<string>();
function handleChange(v: string) {
- setValue(v);
const res = validate(v);
if (res === true) {
setError(void 0);
onChange(v);
} else {
setError(res);
}
}
spark pivot
skill issue idk
austere talon
lolol
jagged reef
lolol
I had to do (message.author as any).globalName 💀
(That's how typing indicator does it)
austere talon
you can also extend the type
import { User } from "discord-types";
interface User {
globalName: string;
}
should work
tidal cloud
man do I love interface overloads
austere talon
it's kinda weird that ts just merges multiple similarly named declarations xd
sly veldt
Cool new update
austere talon
what's your pr
sly veldt
hold on lemme get it
austere talon
you'll figure it out
sly veldt
i glanced over it but it seems overengineered
Gotcha. It seemed very complicated to me as well but I couldn't think of any better way to do it.
sly veldt
ill have to take a closer look another time
Thanks
Whenever works, i'm not really in a rush, i can just keep updating my fork
austere talon
you don't even have to fork technically
check out a normal vencord upstream clone and add it as a user plugin
this way you can enjoy the regular updater with your plugin added
just rename it to avoid conflicts with the original one, name it like MoreUserTags2
sly veldt
Fair, should have thought of that before I forked it lol
But to be fair, I need to fork anyway to have a pr
¯_(ツ)_/¯
I'm fine whatever, it's not a big deal
austere talon
anyway ill take a look soon
sly veldt
Thanks
jagged reef
Looks good? ```ts
import { type Message, User } from "discord-types/general";
interface NameUser extends User {
globalName: string;
}
(UserStore.getUser(userId) as NameUser).globalName;```
austere talon
you shouldn't need to do that
jagged reef
oh
austere talon
import type { User } from "discord-types/general";
interface User {
globalName: string;
}
just this should work
sly veldt
typescript is fun
jagged reef
just this should work
Import declaration conflicts with local declaration of 'User'.ts(2440)
austere talon
that's strange
jagged reef
that's strange
I'm just gonna do ```ts
import type { Message, User as DiscordUser } from "discord-types/general";
interface User extends DiscordUser {
globalName: string;
}```
it's not much longer and it passed checks
austere talon
ya works
summer fiber
how does globalName not exist on User?
was literally just abt to ask abt this, i looked into it abit and vencord is using an outdated version of discord-types, vc uses 1.3.26 and newest is 1.3.3 (https://www.npmjs.com/package/discord-types)
ig globalName was added inbetween 1.3.26 and 1.3.3
its weird though because on the github repo the package.json still says 1.3.26
maby bcuz its in devDependencies i made another project and installed it and it put it in dependencies instead
summer fiber
maby bcuz its in devDependencies i made another project and installed it and it ...
(incorrect)
it seems like its fixed on the github repo
but a new npm thing never got published
who is maintaining this 😭
it can be fixed by doing
pnpm i git+ssh://git@github.com/Swishilicous/discord-types.git
jagged reef
Vee, pretty pwease do this
summer fiber
🙏
summer fiber
Just saying, you might wanna optimize your site for mobile a bit more
yeah i honestly have been putting off fixing the css and everything for mobile 😭
im probably gonna rewrite it at somepoint
summer fiber
omfg it throws a fucking error because theres a comma and not a semi-colon
who the hell is maintaining this package
someone already fixed it in a pr
a pr which is like 8~ months old
i think vee just needs to fork it and use that version
rigid venture
summer fiber
yeah but still annoying
sterile oak
actually its weird that its not a typo and a typo at the same time.
I made an export button for my plugin to export bans. but found out. that Users could have global_name or globalName sometimes
(Don't mind the funky look)
that when I extract them as json. I just directly stringify the bans array in my plugin
no idea why
limber skiff
globalName is on the user class
global_name on the json object
idk if it's you doing the pr but
we should just ditch using that package
we already have typings for so many stuff, adding those and maintaining them would be no problem and way better
sterile oak
we already have typings for so many stuff, adding those and maintaining them wou...
I agree. we should make fluxDispatcher types module to stop repetitive code in plugins
I legit have 3 plugins that have IMessageCreate
lyric comet
the (unofficial) home-manager module for vesktop is complete :)
currently supports enabling plugins by list (more legible for the plugins that dont have settings) as well as both configuring plugins by attrset and by nix convention
any suggestions would be sick
limber skiff
btw just making sure you know, no need to manage the dependencies manually
vencord automatically enables the required dependencies for any enabled plugin
lyric comet
i wasnt sure how it would handle that since the settings.json is readonly so i just defaulted all APIs to true for now (this can be easily overriden)
lyric comet
are deps handled outside of settings/setting.json?
limber skiff
as long you enable fake nitro, message events api is automatically enabled
regardless of the settings config
lyric comet
ah
good to know
ill play around with it a bit before deciding to remove the api defaults
austere talon
who the hell is maintaining this package
noone. it's not maintained
lyric comet
yeah it does seem that required plugins are enabled regardless of config
how does that work? kinda zaney
settings is disabled yet still running with my configs
brazen bone
Alternative would be giving an error if deps aren't enabled, and nobody likes that
limber skiff
yeah it does seem that required plugins are enabled regardless of config
well required plugin should just not be disabled
it ignores the setting
lyric comet
mmm that makes sense
lyric comet
4 on vesktop :)
limber skiff
right
additionally, CrashHandler is enabled by default
however it can be disabled
limber skiff
It's the only plugin enabled by default that can be disabled
austere talon
how does that work? kinda zaney
every plugin that depends on some api lists that plugin in its definition
required plugins state that they are required in its definition
then the plugin loader figures out which plugins are needed as a dependency and which are required and force enables them, regardless of your settings
https://github.com/Vendicated/Vencord/blob/main/src/plugins/index.ts#L39-L67
grave mangoBOT
every plugin that depends on some api lists that plugin in its definition
requi...
**index.ts: **Lines 39-67
export function isPluginEnabled(p: string) {
return (
Plugins[p]?.required ||
Plugins[p]?.isDependency ||
settings[p]?.enabled
) ?? false;
}
const pluginsValues = Object.values(Plugins);
// First roundtrip to mark and force enable dependencies (only for enabled plugins)
//
// FIXME: might need to revisit this if there's ever nested (dependencies of dependencies) dependencies since this only
// goes for the top level and their children, but for now this works okay with the current API plugins
for (const p of pluginsValues) if (settings[p.name]?.enabled) {
p.dependencies?.forEach(d => {
const dep = Plugins[d];
if (dep) {
settings[d].enabled = true;
dep.isDependency = true;
}
else {
const error = new Error(`Plugin ${p.name} has unresolved dependency ${d}`);
if (IS_DEV)
throw error;
logger.warn(error);
}
});
}
limber skiff
let me fix better note box
prime dew
let me fix better note box
ily
limber skiff
uhh I'm not sure what this is supposed to patch
I guess it's this but I could be wrong
I never used this plugin
charred monolithBOT
limber skiff
I guess it's this but I could be wrong
do you remember if it should be before or after the notes? @austere talon
austere talon
tbh idk maybe just delete the plugin
i wrote this so long ago and don't even use it
prime dew
would you know if the padding is supposed to be after or before the notes?
inspect element says it's after the notes
prime dew
uhhhh, I have no idea sadly
limber skiff
okay I see it's after
prime dew
I'm using a theme that also hides the category dividers so I can't really tell
limber skiff
I'm using a theme that also hides the category dividers so I can't really tell
okay but did you notice a different after the thing broke?
I might just remove it
prime dew
okay but did you notice a different after the thing broke?
well yeah
limber skiff
what was it
limber skiff
the heading?
prime dew
yes
limber skiff
oh wait
austere talon
It already adds the lastSection class if it is the last
i think the fix is removing the margin
if the note is hidden
maybe im wrong
anyway can it be hidden with css
is the plugin even needed
prime dew
let me disable the plugin and try again with the css they sent me in #🎨-theme-development
cunning canyon
it was something like return (hideNote) ? null : (the note box) when i wrote it
and the patch change null to some margin when it is last section
prime dew
let me disable the plugin and try again with the css they sent me in <#113484432...
yup, it worked
using this #🎨-theme-development message
prime dew
the "Note" thingy
limber skiff
they may have removed hideNote completely
nvm they didnt
but they no longer hide the heading
prime dew
but yeah, if nobody uses it and it can be hidden with css you can just remove it
limber skiff
no
prime dew
austere talon
<:vennieflower:1216987042362298450>
why is that spotify card so fucked up
prime dew
why is that spotify card so fucked up
oh, that happens sometimes
I just have to restart discord
it's because I have a css snippet with emojis instead of icons
restarting discord fixes it
prime dew
the "Note" thingy
here it's normal
limber skiff
I fixed better notes
prime dew
limber skiff
I'm dumb nvm
limber skiff
better notes is a crazy plugin
it's so dumb and it has one of the craziest patches
lyric comet
every plugin that depends on some api lists that plugin in its definition
requi...
oh sick thanks
charred monolithBOT
prime dew
it's so dumb and it has one of the craziest patches
And probably the smallest userbase
austere talon
idk it's very niche and not that useful tbh
but it's also very old
old plugins didn't have as high quality standards
verbal pumice
old plugins didn't have as high quality standards
fart v2
summer fiber
noone. it's not maintained
kinda is what i was thinking
limber skiff
better settings is hot wtf
I actually tried to remove the transition in the past but I only tried for like 2 hours and couldnt get it
and never ended up trying again
lmao
limber skiff
my first tries were super simple
brazen bone
Yeah I did some similar things at first
charred monolithBOT
mint thistle
it explodes 
austere talon
what is this for again? @jagged cloak removing mentions?
grave mangoBOT
https://github.com/Vendicated/Vencord/blob/ca18b6e0446b6f1858a82de47489bb01f925c...
**index.ts: **Lines 286-288
function cleanMessage(content: string) {
return content.replace(new RegExp("<[^>]*>", "g"), "");
}
@austere talon what do you think?
austere talon
<@343383572805058560> what do you think?
that's still not good because if there's an error it will skip setting state
spark pivot
should i move the onChange call then?
mortal fractal
**index.ts: **Lines 286-288
```ts
function cleanMessage(content: string) {
r...
this should be more specific because if its only for mentions itll fuck up unembededd links, emotes, and anyone just posting <>
if this is just for notifications it should be using discord's own notification content function
(findByProps("makeTextChatNotification"))
limber skiff
@spark pivot ohh is that the reason match doesnt update when you paste full patch
I was debugging this rn lmao
I'm modifying the patch helper cuz adding support for array finds
spark pivot
cool
spark pivot
should i move the `onChange` call then?
not sure how to fix match not inputting, but this change might fix it
charred monolithBOT
stark flint
LOL
limber skiff
lets goo
spark pivot
i think github is being bombed with these issues right now cause i searched for this and all of them are less than 5 minutes old
there's like ten per second
jagged cloak
https://github.com/Vendicated/Vencord/blob/ca18b6e0446b6f1858a82de47489bb01f925c...
i honestly dont remember so i yeeted it lmao
stark flint
what's the point if literally no one else can hear it
lime stone
you can already do this lol
limber skiff
the available patch is nice
spark pivot
wait no the second option should definitely be apart of fakenitro
stark flint
can you not just use the sound preview button to listen to them though?
or is that also blocked for sounds you don't have access to
spark pivot
I think it is
stark flint
unfortunate
charred monolithBOT
charred monolithBOT
turbid hatch
i dont really understand what valuepointer is for
like it's just making a constant mutable
just make it mutable in the first place
you're already using typescript, it's type safe and will complain if you assign the wrong value, your pointer object is noise
median rapids
does that second patch really just fucking work lol
median rapids
they def would
good chance they accept this in https://discord.com/security with the shit they been accepting lately lol
rugged spire
i would add it anyway, discord has their "totally-forever-in-beta soundmoji feature" and it is locked in that UI
frail skyBOT
median rapids
i can surely make discord break
so eel
frail skyBOT
rugged spire
does this user know what venmic is
charred monolithBOT
glass jasper
so if that CharCounter is garbage is someone going to make a new/better one?
because that is actually a very good plugin idea
jagged reef
so if that CharCounter is garbage is someone going to make a new/better one?
Why?
It’s kinda useless
glass jasper
Why?
A charCounter is always a good plugin especially for people whom don’t have nitro and are always writing out long sentence that get close too the 2000 minimum characters non nitro people have
jagged reef
If you need to know the number of characters in smth, just copy it and go to https://wordcounter.net/character-count
jagged reef
A charCounter is always a good plugin especially for people whom don’t have nitr...
Discord already pops up a tooltip that says how many characters you have left when you get close
glass jasper
If you don’t wanna make it then don’t. I’m not going to have this convo, I just think some people especially the average Vencord user (dumb as a bag of rocks, and 2 minutes reading the support channel would prove that statement correct) would find it useful.
jagged reef
💀
glass jasper
why the waffling
mb for not being good at english and only having the shitty American education system to teach me stuff
I’m sorry, I just voice my opinion that i thought it’s a pretty good plugin and that some people (especially the not that tech smart people) would find it useful. Yes it’s niche and but there is a small group of people that would find it useful. Plus there are a lot of niche and weird stuff in Vencord. So a charcounter plugin wouldn’t be that weird for being a thing
charred monolithBOT
charred monolithBOT
crude hearth
@austere talon merge my pr
lime stone
never merge manti pr
charred monolithBOT
rugged spire
i will unfuck vendroid
charred monolithBOT
dusk blaze
i will unfuck vendroid
what if we start vendroid fork together
rugged spire
i think i will pretend to know what i am doing for a little while first
rugged spire
dusk blaze
average vee code
rugged spire
yea
charred monolithBOT
rugged spire
is the java side even implemented anywhere
lime stone
oh god. java? be prepared for all of my eyes on this code
(what's the expression? how many eyes do i have?)
rugged spire
SHITCODING
i have never programmed an android app
i am pretending to know what i am doing
if anyone sees this i am aware that i will be shot
or husked
idk whats worse
shy veldt
does that second patch really just fucking work lol
i swear i tested it
and the req returned some error code
lol
it just works????
thats stupid
how did it not work before
brazen bone
i am pretending to know what i am doing
That's what everyone does
inland musk
about the prospect of enabling /vencord-debug globally, would it be better just to have debug info copy to clipboard upon clicking versioning instead like how bd has it
just figure it might be good to keep the same system
austere talon
is the java side even implemented anywhere
wdym
rugged spire
of that onBackPressed() method
crude raptor
it would be nice if all the prs it wants to be merged were
dusk blaze
yop
crude raptor
🙏 improved textreplace
🙏 tone indicator meanings
austere talon
btw @rustic nova multiple complained that PinDms no longer sorts by most recent message
ik it's a setting but maybe we should change the default value of it to enabled
or rather make that sort order the default
grave mangoBOT
https://github.com/Syncxv/Vencord/blob/fix-pin-dms-and-improvments/src/plugins/p...
**index.tsx: **Lines 43-51
pinOrder: {
type: OptionType.SELECT,
description: "Which order should pinned DMs be displayed in?",
options: [
{ label: "Most recent message", value: PinOrder.LastMessage, default: true },
{ label: "Custom (right click channels to reorder)", value: PinOrder.Custom }
],
onChange: () => forceUpdate()
},
oh i didnt realise you already did that
charred monolithBOT
charred monolithBOT
Update: I added both features. Here's what it looks like:
(You can click out of the box/click the X to dismiss it)
I tried my best to make it look as good as possible but let me know if you guys have any suggestions.
swift fjord
unfix it
shy veldt
bros pushed a quick one
rugged spire
bros pushed a quick one
i have no idea what this does but they should NOT be doing that like that
austere talon
it's auto generated
rugged spire
o is this the datamining changes or something
brazen bone
i have no idea what this does but they should NOT be doing that like that
That's just defining the module exports
rugged spire
That's just defining the module exports
I knew that...
rugged spire
wtf
charred monolithBOT
sterile oak
how does VesktopNative/VencordNative work? I want to see if my skillset allows me to create extensionNative
brazen bone
They do some manner of ipc with the electron host process
charred monolithBOT
sterile oak
finally, I feel my plugin is ready for pr. but I am nerves for some reason
austere talon
just do it
still wasp
just do it
Not sponsored by nike
glass jasper
just do it
Shia LaBeouf
charred monolithBOT
long stream
@spark pivot im really confused :/
shoudl i still try to find by the string constant? or should i use another prop instead?
austere talon
<@523338295644782592> im really confused :/
the issue is that you're relying on the variable name I
long stream
ye i get that
austere talon
any variable name that is only a few random letters is auto generated and you shouldn't rely on it
.Messages.FOO_BAR is discords name so you can rely on it
long stream
im just not sure on how i should be finding it if not by that
austere talon
for example by using find: ".Messages.FOO_BAR"
long stream
i already did that here but it didnt really work out
austere talon
yeah you need to find something else that is unique then
brazen bone
Messages.AVATAR_UPLOAD_EDIT_MEDIA seems unique
austere talon
use ctrl shift f to search for it and make sure there is only 1 result
long stream
yeye im already doing that xd
brazen bone
Localization strings (Messages.FOO_BAR) and exports (fooBar:function() are often good finds
long stream
`Messages.AVATAR_UPLOAD_EDIT_MEDIA` seems unique
aaaa i completely missed this
i still dont really understand how the patch system works 
austere talon
brazen bone
i still dont really understand how the patch system works <:BocchiPain:107250510...
It's just find a string that uniquely identifies the module you want to patch, and then regex replace things in that module
limber skiff
in fact I think I never used a export as a find in my patches
brazen bone
Yeah class names are decent too
brazen bone
Yes absolutely
limber skiff
banger find for context menu api
long stream
ye i saw that while i was looking thru other plugins xd
brazen bone
because discord can mangle in the future
I'm not cynical enough for that mindset
limber skiff
I'm not cynical enough for that mindset
because you were not here when everything was mangled
brazen bone
Yep
limber skiff
and honestly, it was not that bad
long stream
also btw @brazen bone do u wanna be credited for helping ? i feel kinda rude for not doing it
brazen bone
also btw <@236588665420251137> do u wanna be credited for helping ? i feel kinda...
Nah
limber skiff
the hardest thing was deciding how to name things lmao
long stream
oke 
limber skiff
how do I call this fucking export 😰
brazen bone
How would findByProps work? Isn't the whole problem that those props are all mangled?
limber skiff
it would try the filter on every export too
brazen bone
Oh
limber skiff
so you would match the props on the exported objects
not the actual export name
and findByCode was the way for components/other methods
brazen bone
I'm still annoyed that findByCode searches in the module's exports and not the module itself
austere talon
I'm still annoyed that findByCode searches in the module's exports and not the m...
findModuleId is what you're looking for
findModuleId("renderSingleLineMessage:function()")
limber skiff
I'm still annoyed that findByCode searches in the module's exports and not the m...
because it's literally meant to find an export not the module
btw ven do we have a force updater api for messages already?
I think I wanna find again how to manually force updates, but without wrapping the component
brazen bone
I guess in cases where exports are mangled, getting the raw exports dict is not particularly useful
rugged spire
btw ven do we have a force updater api for messages already?
stupid question, why would you want to do that
limber skiff
there are many cases for wanting to force a re-render in a message
rugged spire
examples?
limber skiff
okay I cant think of one rn but in the past many have wanted that
I dont understand why are you husking
it's not a crazy thing to want?
brazen bone
It goes against everything about how React is meant to be used
rugged spire
It goes against everything about how React is meant to be used
Vencord abuses React
I only know how to use React because of Vencord and I can tell half of it is done super jank manual force rerendering
austere talon
stupid question, why would you want to do that
for example to update attachments, message content, etc
brazen bone
If you want to update the message, update the MessageStore and any useStateFromStore will automatically be rerendered
austere talon
dispatching message updates is a bit painful because it expects an api message
while we're working with "model" messages
rugged spire
while we're working with "model" messages
wdym model messages
austere talon
api message is raw api data that contains things in snake_case and such
rugged spire
oh yes
limber skiff
It goes against everything about how React is meant to be used
Vencord does not follow how things are supposed to be used
austere talon
discord serialises that into a message class that uses camelCase and has methods and such
rugged spire
Vencord does not follow how things are supposed to be used
Welcome to hacking your own code on top of other people's propietary bullshit app
austere talon
to dispatch a message update event we basically need to serialise messages to an api message which is kinda complicated
rugged spire
You will enjoy your stay
rugged spire
to dispatch a message update event we basically need to serialise messages to an...
wtf
austere talon
otherwise it breaks attachments and embeds
rugged spire
are attachments deserialized in a weird way
austere talon
?
rugged spire
It isn't just some renaming to camelCase for the actual model message
austere talon
anyway message updating is for example needed for the HideAttachments plugin
rugged spire
There's some kind of attachments class or something, right?
rugged spire
anyway message updating is for example needed for the HideAttachments plugin
i forgor if that is a plugin in mainline vencord
yop
how... does that plugin even cause a rerender
does it?
austere talon
i don't want to be mean but you're not making any meaningful contributions to the discussion
rugged spire
true
I'm just curious (that's all)
btw i've recently been doing horrible patches with vendroid
austere talon
btw ven do we have a force updater api for messages already?
anyway no there isn't
i experimented a bit with it but didn't got anything to work
there's https://github.com/Vendicated/Vencord/tree/componentUpdaterAPI but it's not good
it basically just patched the message component to insert a useForceUpdater and store it in a global map
limber skiff
there's https://github.com/Vendicated/Vencord/tree/componentUpdaterAPI but it's ...
alright I will try more later then
brazen bone
If I'm reading the code correctly, you should be able to dispatch an empty LOAD_MESSAGES event and every useStateFromStores([MessageStore]) will recheck
limber skiff
trigger the state re-render
brazen bone
You mean you want the useStateFromStores to cause a rerender even if the returned value is the same?
limber skiff
I want to invoke the re-render of the state without the store changing
brazen bone
LOAD_MESSAGES does not change anything in the store ```js
LOAD_MESSAGES: function () {
return true;
},
It just — for some ungodly reason — reports that the store was changed, and thus all change listeners are run
There's no other listeners for LOAD_MESSAGES either, though it does feel a bit fragile
austere talon
rerendering all messages is likely not a good idea
brazen bone
Only those where the useStateFromStores callback differs
Discord already does that on every messagestore change
Though I do admit that I haven't tried whether it actually works
charred monolithBOT
charred monolithBOT
spark pivot
zWHAT
charred monolithBOT
charred monolithBOT
glass jasper
HUH!
golden gulch
austere talon
why alt backspace of all things
we really need a better way to do keybinds for plugins
i personally think that keybind is really bad but i do like the general idea of the plugin, i thought about a mention toggle keybind as well
austere talon
we really need a better way to do keybinds for plugins
like some api where users can just change it
golden gulch
why alt backspace of all things
backspace because it's what the old powercord plugin used, alt so that it didn't conflict with other things
I'm fine with other keybinds this is just the one that was easy to find and didn't conflict with anything
a keybinds API would be great though yeah
austere talon
well any keybind is gonna be subjective and someone is gonna be unhappy xd
it's the same with the quick delete plugin
austere talon
we really just need a way to change it
jagged cloak
i swear this was brought up b4
cunning canyon
it was arjix's pr
and ofc closed
golden gulch
backspace because it's what the old powercord plugin used, alt so that it didn't...
for context, the powercord plugin would toggle mention if you pressed backspace and the message bar was empty
or something similar to that, I don't remember it exactly
yeah
charred monolithBOT
charred monolithBOT
rugged spire
yall sre reminding me to fix that double click to edit/reply plugin to not edit the message
glass jasper
@austere talon i’m probably about to fuck this up but i tried, don’t get mad at me because i don’t completely understand how github works yet
glass jasper
ok there we go, don’t blame me if this was bad or fucked up it was first time using github like that
austere talon
<@343383572805058560> i’m probably about to fuck this up but i tried, don’t get ...
no you did it perfectly!!
glass jasper
no you did it perfectly!!
ok :3
i’ll see about this using a class instead of the selector
austere talon
in js you need to assign a class to the button element
glass jasper
i figured, it’s just using the selector is how i was taught how to do it
stark flint
probably should've PR'd to dev but bleh
charred monolithBOT
spark pivot
probably should've PR'd to dev but bleh
You can change it
Oh it isn’t your pr
Silly me 
stark flint
lol it is
spark pivot
Even sillier me
stark flint
charred monolithBOT
charred monolithBOT
glass jasper
@austere talon ok i think i did
( @bleak gyro help me out with it)
austere talon
please actually test your changes before pring
charred monolithBOT
austere talon
both me and <@721717126523781240> did
that's not possible because the code will not compile
it contains a syntax error
glass jasper
it contains a syntax error
FUCK ME, it’s that fuckin typo i fixed before why is it back
spark pivot
There were two syntax errors lol
glass jasper
There were two syntax errors lol
fuck, i fix one typo with that thing and another one shows up
goddamn
spark pivot
Are you using vscode
surreal fiber
offical
rancid dragon
nvhhr is trying to say you spelled it wrong
- name: "krystalskulloffical",
+ name: "krystalskullofficial",
I didn’t even notice either
glass jasper
god hates me and doesn’t want me in the constants.ts
glass jasper
@austere talon ok idk what to do, i’m started to angy, i’ve never used github before so i’m trying
spark pivot
You’re good at this point
The pr is ready for merging
If you need any more help with github let us know
glass jasper
If you need any more help with github let us know
ok thanks ☺️
glass jasper
it still hasnt merged
same
crude raptor
it will never happen,,, vencord is actually dead and we're all hallucinating
glass jasper
i’d believe it
spark pivot
it still hasnt merged
I had one of my prs get merged recently
That brings the total to 2
mint thistle
it still hasnt merged
same 2
gritty iris
it will never happen,,, vencord is actually dead and we're all hallucinating
whats vencord??
charred monolithBOT
stark flint
ok this is apparently broken and only shows on my main client hmmmm
glass jasper
ok this is apparently broken and only shows on my main client hmmmm
gatekeeper
stark flint
ok this is apparently broken and only shows on my main client hmmmm
- buttons: buttons.map(v => v.label),
+ buttons: buttons.length > 0 && buttons.map(v => v?.label) || undefined,
this seems to work but it feels ugly.. and i highly doubt it's the best approach
shy veldt
and another one
well, well, well
spark pivot
Joyous
median rapids
well, well, well
wtf they're naming stuff??
austere talon
just moving from default exports to named exports
why is this so surprising to you
glass jasper
imma move what channel i post that in
median rapids
why is this so surprising to you
the fact it's gradual is
glass jasper
well i’m done, maybe this will be merged maybe it won’t ig we’ll see
glass jasper
bruh
mint thistle
bleeh
median rapids
well i’m done, maybe this will be merged maybe it won’t ig we’ll see
real
charred monolithBOT
glass jasper
real
i mean it’s like actually good (i’m not just saying that because I made it, and i think it’s good) it really is because it does something that should’ve been done a long time ago
Vee is the whole reason, I did all this. I was originally just going to post it as a #🎨-css-snippets that people could just ”@import” they wanted it but they talked me into making it a PR and doing all this other work
charred monolithBOT
I have implemented the feature but it does require a manual reload. is there any way I can show a reload prompt.
charred monolithBOT
charred monolithBOT
charred monolithBOT
charred monolithBOT
charred monolithBOT
glass jasper
sad moment for us canary user
prime dew
I guess the 2nd one is related to the SilentTyping plugin
glass jasper
I guess the 2nd one is related to the SilentTyping plugin
I think it applies all the buttons on the chatbar, because I main canary and the only ones here is the default badges
prime dew
hurray
glass jasper
so all char bar buttons gooone
yeah that’s what i figured
brazen bone
vee has a life outside of vencord
Doubt
austere talon
rude
glass jasper
ok i’m sorry I was just reminding about it
glass jasper
maybe Vee working for Discord was true and they’ve been slowly changing code to be more helpful to the client modding
austere talon
yep
doing it slowly one change at a time to make it harder for them to revert if they ever notice
median rapids
> vee has a life outside of vencord
Doubt
real
charred monolithBOT
MessageLogger is useful but takes a lot of place. More often than not, deleted and edited messages are more of a curiosity than critical information, and often messages are deleted for a good reason, so keeping them fully visible is counterproductive.
I don't think this is quite ready to be...
spark pivot
prime dew
Every day it gets longer
What's the last section for, by the way?
glass jasper
bruh, please be nicer to us canary user
umbral hedge
Discord just bumped stable from Electron 22 to 28 :3
lime stone
cool
umbral hedge
and 64 bit?
idk i only use 64bit so