yeah I did
#๐พ-core-development
limber skiff
austere talon
IS_STANDALONE && !IS_WEB ? gitTag
limber skiff
that works lol
austere talon
just make sure pnpm buildWeb and pnpm buildWeb --standalone still compiles
austere talon
just make sure pnpm buildWeb and pnpm buildWeb --standalone still compiles
(also maybe make sure that you're not including random dead code in them)
but u didn't so all good
umbral hedge
real
charred monolithBOT
charred monolithBOT
charred monolithBOT
limber skiff
I just realized we will not need the git-tag
austere talon
ye
austere talon
There are 1 update
limber skiff
dumb prettier is formatting the file
limber skiff
how else can I do it
austere talon
- shcEmojiComponentExport=($self.setEmojiComponent($<component>),void 0)
+ $self.setEmojiComponent($<component>)
?
im confused what thats supposed to do
limber skiff
its like this
var a = function() {}
, shcEmojiComponentExport = (bla(a),void 0)
, c = "asdas"
it's just to execute that function when declaring the var
austere talon
why not just change it to
var a = $self.setEmojiComponent(function() {})
and make the setter return the value
limber skiff
not really friendly if any other plugin wants to touch that
limber skiff
not really friendly if any other plugin wants to touch that
the current way wouldnt mess up any other plugin
plus garbage collector would throw that variable away anyways
austere talon
hmm fair
unborn garnet
I can't believe he just copied me
austere talon
owned
frail sky
quick ibex
Is this real life wtf
cunning bobcat
not when keywordAlerts is merged
cunning bobcat
wheeze
woeful sable
Discord Errors
1
honest geyser
discord โ
woeful sable
there are discord employees here and they use vencord
limber skiff
it was the quantifier wrong
honest geyser
there are discord employees here and they use vencord
that's possible actually
honest geyser
no I mean that discord emlpoyees uses discord
woeful sable
discord employees sure use discord
woeful sable
honest geyser
discord employees may use venchord
charred monolithBOT
limber skiff
yikes
cunning bobcat
SO THAT'S WHY I WAS HAVING ISSUES WITH CHANNEL MENTIONS
lament nimbus
@dusky laurel ๐ธ
dusky laurel
limber skiff
very descriptive title
charred monolithBOT
limber skiff
why the heck does regex engine start at the lookbehind
why not do the match and then check the look behind ๐ญ
quaint sapphire
You can always make your own regex engine and realize how fucking hard it is to make it efficient and in the way you want.
limber skiff
thanks for the useless comment
woeful sable
why ur being passive aggressive for no reason
quaint sapphire
Coping is the only true answer when regex doesn't work due to engine limitations
quaint sapphire
Now make ur own engine optimized for your specific regex to bring that time down to 0.1 ms
woeful sable
what
quaint sapphire
You love
No joke tho, making a regex engine is a nice learning experience
Full with mental and physical pain
quick ibex
@umbral hedge can you close #573 as won't be resolved?
umbral hedge
ยฏ_(ใ)_/ยฏ
quick ibex
Whatever
quick ibex
Mfw org isn't owned by ven iirc
charred monolithBOT
umbral hedge
well i suck anyways so 
umbral hedge
My u key not working on mobile tf
trolled by keyboard
quick ibex
well i suck anyways so <a:shiggy:1024751291504791654>
Just to have faster response time to stuff like issues ig
austere talon
Mfw org isn't owned by ven iirc
it is
austere talon
apparently ven removed my repo perms
I removed it when you merged a pr I specifically said should be included in a different commit
and like you really only ever used it to make a pr then immediately merge it so it was kind of pointless anyway
umbral hedge
removed for merging a docs change so epic
also what you said at the time didn't really make any sense since it was a commit. you meant a different PR.
somber ginkgo
bye megu
charred monolithBOT
What should the plugin/feature do?
Prevent DMs from being automatically closed when using the "Report Spam" button at the top of the channel.
How does it work?
- Enable plugin
- Report a DM using the "Report Spam" button at the top of the channel
- The channel stays open after submitting the report, instead of closing automatically
Any images or mockups?
**Possib...
azure jay
austere talon
charred monolithBOT
austere talon
btw I invited you too @limber skiff
limber skiff
omg
charred monolithBOT
jagged cloak
balls gaming
tidal cloud
cookies
turbid hatch
๐ญ
austere talon
wha
turbid hatch
oh i meant that in response to tasky
i want cookies now
wait i can get them from greggs
score
tidal cloud
I'll make you some cookies
turbid hatch
oh yes
i feel like im saying cloud integrations too much
i could change the second one to "if you have not yet set them up"
which will be semantically the same but with less repetition
does that sound good?
jagged cloak
Vencord powered by ai
somber ginkgo
i cant wait to get made fun of
austere talon
that extension won't work on Firefox because it's not signed, it's a zip not a crx which chrome might dislike, you include the update url unconditionally which might break submission to stores
but most importantly the extension won't be updated unless version is bumped
which will be in sync with chrome and Firefox store updates
somber ginkgo
it's fine there's just some things missing
i forgot renaming branches closes prs my bad
and i have no experience with this build process so i would appreciate some help 
charred monolithBOT
austere talon
lol no need to accuse people of lying
that person probably just assumed it would store permanently
because the BD one does
quick ibex
yea
@austere talon can you run that thing to check the patches??
Discord crashes when I join vc
Can't figure out source of problem
nvm
all i had to fucking do was restart my pc
austere talon
if a patch breaks it won't crash
it'll just crash when a patch still applies but smth goes wrong
charred monolithBOT
austere talon
please do not use DevilBro plugins as inspiration 
somber ginkgo
when are we getting a memory leak plugin for people with betterdiscord withdrawal
quick ibex
when are we getting a memory leak plugin for people with betterdiscord withdrawa...
there was a pr for it made by @high oracle
but it didnt get merged
dunno why ...
somber ginkgo
@austere talon feel free to reject 578, unless you have future plans for it
because i am extremely 
cunning bobcat
oh yeah I'm clueless about svgs & shit for keywordAlerts, so if anyone has any pointers or even a good built in svg i would appreciate a comment on the pr
austere talon
<@343383572805058560> feel free to reject 578, unless you have future plans for ...
Well I'm not planning to work on it cause I don't really see the point
pretty sure we could just make a second extension listing for nightlies
@umbral hedge im confused, why did you always pr into merge instead of just directly pushing
the require pr is off so technically you should have been able to just push?
(and I can push fine without pr)
somber ginkgo
pretty sure we could just make a second extension listing for nightlies
simply just push updates for a second extenson for every commit
cunning bobcat
I'm asking my past self the same question
umbral hedge
<@545581357812678656> im confused, why did you always pr into merge instead of j...
we discussed this before
it has a "must pass tests" requirement
so only owners can bypass that
austere talon
happens so much
idk what they cooking
wait
we could make the workflow restart itself if it's a 500 maybe
might be related
charred monolithBOT
turbid hatch
@spark cove
austere talon
nookies addicted to refactors
crude hearth
@austere talon Type '{ placeholder: string; options: { label: "EST" | "Etc/GMT" | "Etc/GMT+1" | "Etc/GMT+10" | "Etc/GMT+11" | "Etc/GMT+12" | "Etc/GMT+2" | "Etc/GMT+3" | "Etc/GMT+4" | "Etc/GMT+5" | "Etc/GMT+6" | ... 367 more ... | "Pacific/Wallis"; value: "EST" | ... 377 more ... | "Pacific/Wallis"; }[]; isSelected: (tz: any) => boolean;...' is not assignable to type 'IntrinsicAttributes & { placeholder?: string | undefined; options: readonly SelectOption[]; value?: SelectOption | undefined; look?: 0 | 1 | undefined; ... 23 more ...; "aria-labelledby"?: boolean | undefined; } & { ...; }'.
limber skiff
nookies addicted to refactors
shhhhhhhh
turbid hatch
<@343383572805058560> Type '{ placeholder: string; options: { label: "EST" | "Et...
not all the options are compatible with searchableselect
some have different names or dont exist at all
crude hearth
not all the options are compatible with searchableselect
I have no idea what you said but got it to work by changing "select" into "onChange"
turbid hatch
yeah that sounds about right
they changed the names of props for implementing the new select component
which i assume you're using now
austere talon
didn't we already go over this just don't provide timezone autocomplete on old electron
turbid hatch
crude hearth
guhhhh 95% of users are in old electron
turbid hatch
you could cache the array in indexeddb if you really wanted
austere talon
explain use case for timezone picker
turbid hatch
here we go again
crude hearth
when would I need to pick a timezone ever with the plugin
for other people ๐คจ
example: other person you know doesnt use tzdb
austere talon
why wouldn't those other people just use the plugin
crude hearth
but you know what his timezone is
austere talon
and register
austere talon
wha
crude hearth
why wouldn't those other people just use the plugin
idkk
but autocomplete select works good
I just need to fetch timezones when bencord opens and cache them
@austere talon @tame escarp write code for m
austere talon
crude hearth
actually
turbid hatch
its like
crude hearth
you know what
crude hearth
it will fetch everytime vencord is open
turbid hatch
no
crude hearth
yes
tame escarp
<@343383572805058560> <@714583473804935238> write code for m
Only if you accept Rust submissions
quaint sapphire
Manti is so lazy
tame escarp
when are we making rust wasm bindings to the vencord api tbh
austere talon
I still feel like you're working on a non existent use case
like you can just get the current users current timezone and give it to them if needed, but that should just automatically be handled by the plugin. You wouldn't need to pick timezone for other users cause they just use the plugin (or if they don't use it they would know their timezone, or if they don't then the picker wouldn't help you much)
turbid hatch
you dont need to worry about indexeddb, datastore api abstracts it away for you
austere talon
like im just trying to understand the use case, it doesn't seem real to me
turbid hatch
the tsdocs that are there should be able to help you understand how to use it
quaint sapphire
I still feel like you're working on a non existent use case
like you can just g...
No
turbid hatch
'no'
crude hearth
I still feel like you're working on a non existent use case
like you can just g...
yes but the problem issssss
solemn spoke
sorry went too late
crude hearth
not all people wanna go to timezonedb website and register
solemn spoke
whats going up
quaint sapphire
The problem is that manti is too lazy
austere talon
not all people wanna go to timezonedb website and register
do you have any reason to believe that?
crude hearth
I wouldnt wanna ask anyone to register into t imezonedb
quaint sapphire
do you have any reason to believe that?
Yes
solemn spoke
The problem is that manti is too lazy
pr the pr and make it better tho
austere talon
I wouldnt wanna ask anyone to register into t imezonedb
then what's the point of having an api
turbid hatch
if i was given a nudge saying "hey! go to timezonedb to register yourself" i would probably do it
austere talon
quaint sapphire
I know a lot of people that don't give a shit about stuff like this
crude hearth
then what's the point of having an api
for people who use the plugin
for them it makes it easier
for people who dont you can enter manually
austere talon
you're saying you don't want to ask people to use the api though
tame escarp
I wouldnt wanna ask anyone to register into t imezonedb
delete system32 / /usr / /System if they dont
quaint sapphire
you're saying you don't want to ask people to use the api though
Yes
unborn garnet
when chat bar icon api
turbid hatch
this feels like you're creating a problem by not wanting to implement a solution
solemn spoke
if i was given a nudge saying "hey! go to timezonedb to register yourself" i wou...
eh, i mean, it would be better to ask for the person'd timezone and add yourself instead of asking them to register on some random website
crude hearth
you're saying you don't want to ask people to use the api though
its just ... a bit of work
I know very lazy people
austere talon
for people who dont you can enter manually
and how would you get that person's timezone? they would just tell you current time or the timezone, in first case a picker doesn't help at all, in second case you would already have their timezone so you can just paste it
solemn spoke
honestly
turbid hatch
i cant believe a fucking timezones plugin has had the most development drama on this project
quaint sapphire
Why should I ask someone else to register to smth that will only be useful for me?
What will that other person gain?
That other person might not wanna make their timezone public as well
crude hearth
and how would you get that person's timezone? they would just tell you current t...
if its someone I know I would know in which timezone they live
unborn garnet
and how would you get that person's timezone? they would just tell you current t...
your pfp kinda looks like an nft
austere talon
lie
quaint sapphire
i cant believe a fucking timezones plugin has had the most development drama on ...
Meanwhile all my other plugins that didn't get merged: https://cdn.discordapp.com/emojis/810291471218638888.png?quality=lossless&size=48
crude hearth
gmt +1
turbid hatch
okay
you know what
fuck it
just let people input a fucking offset
there
no need to faff with timezones
crude hearth
just let people input a fucking offset
YES THEN WHEN IT BECOMES WINTER
austere talon
no
crude hearth
IT WILL SHIT ITSELF
turbid hatch
THEN YOU CAN CRY!
solemn spoke
honestly
i tried to implement my own time db into the first timezone plugin done. it was a horrible implementation since i would need to check if the user is registered on timezonedb and if not check on the local db
turbid hatch
DECIDE HOW YOU WANT TO RESOLVE THIS!!!
quaint sapphire
Why are you even having this convo?
crude hearth
DECIDE HOW YOU WANT TO RESOLVE THIS!!!
THERE IS LITERALLY NO OTHER WAY TO SOLVE THIS
solemn spoke
i tried to implement my own time db into the first timezone plugin done. it was ...
if it done locally then the user will need to wait to it to fetch for no reason
quaint sapphire
THERE IS LITERALLY NO OTHER WAY TO SOLVE THIS
Shut up manti
crude hearth
I dont know
quaint sapphire
okay whats the solution
Please don't waste ur time with manti
quaint sapphire
There is no issue
crude hearth
whats the issue
basically ven doesnt want me to have a option to set users timezone
quaint sapphire
Manti is just lazy
quaint sapphire
basically ven doesnt want me to have a option to set users timezone
No that did not happen, ven is just questioning how useful that is
austere talon
basically ven doesnt want me to have a option to set users timezone
that's wrong, I want you not to add a limited or faulty multiple choice if there's better solutions
crude hearth
No that did not happen, ven is just questioning how useful that is
questioning and telling me that its useless
solemn spoke
basically ven doesnt want me to have a option to set users timezone
as in manually setting up a user timezone?
austere talon
I think if you really think it will be a common use case to add other people's timezones, then you should just make a website people can open and it tells them their timezone
crude hearth
that's wrong, I want you not to add a limited or faulty multiple choice if there...
well whats better solution
turbid hatch
okay, lets see the options:
- ship a fetch request to a list (with cache or it probably won't ever get an approval)
- making sure you have the native method in so the fetch isnt used if its not needed
- make everyone sign up to timezonedb
- make everyone go somewhere else so the list isnt necessary
- offsets
these seem like the most plausible solutions, if anyone has anything else do let me know
quaint sapphire
Ok manti, can you shut the fuck up?
I'll keep updating the plugin ffs, you won't do shit
austere talon
what's that timezone standard called again
crude hearth
utc?
quaint sapphire
Etc?
solemn spoke
okay, lets see the options:
- ship a fetch request to a list (with cache or it p...
cant we just check the local timezone list then the website?
quaint sapphire
I am working
Wow, that's a lot of work
turbid hatch
why would you need the local list and* the website
crude hearth
Wow, that's a lot of work
yeah
quaint sapphire
cant we just check the local timezone list then the website?
That's the plan
solemn spoke
why would you need the local list and* the website
because i wouldn't like to sign for a random ass website
that's ehy
quaint sapphire
If electron is old then we fetch
austere talon
I think if you really think it will be a common use case to add other people's t...
@crude hearth
crude hearth
If electron is old then we fetch
which is 95% of the time
austere talon
this is way more robust than making people ask idiots what their timezone is
crude hearth
I think if you really think it will be a common use case to add other people's t...
guhhh
turbid hatch
if you're really going that method then you need to implement the native method as priority
quaint sapphire
this is way more robust than making people ask idiots what their timezone is
Sometimes you just know the timezone of the other person
austere talon
literally just make super simple site whatismytimezone.mantika.dev that shows timezone and copy button
turbid hatch
wait
austere talon
Sometimes you just know the timezone of the other person
then you just paste it into the box?
austere talon
all im saying is that adding hardcoded auto complete is stupid
quaint sapphire
then you just paste it into the box?
Please don't repeat this ven
austere talon
because it will be incomplete or have options your client doesn't actually support
turbid hatch
literally just make super simple site whatismytimezone.mantika.dev that shows ti...
document.innerHTML = Intl.DateTimeFormat().resolvedOptions().timeZone;
crude hearth
all im saying is that adding hardcoded auto complete is stupid
well in that case they can use ETC/Gmt+3 options
thats why it exists
solemn spoke
I think if you really think it will be a common use case to add other people's t...
too much work for not much imo. it works well enough to ask to it instead of them opening for a random ass limk
turbid hatch
i think that works in stable btw
crude hearth
if city is not in the list
quaint sapphire
This plugin should not expect or assume that any other person in the world is willing to give you the timezone string as is
crude hearth
theres nothing we can do
turbid hatch
https://discord.coffee/ABXdTfR.png yeah it does
austere talon
Auto complete is bad if you hardcode it because incomplete or invalid options and it's not user friendly because noone will know their timezone properly or if they do you won't need autocomplete cause you just copy paste it in
simple solution is to only have auto complete if you have actual Intl data and make the simple what is my timezone website like I said that lets anyone get their timezone easily
turbid hatch
so the website isnt necessary
solemn spoke
This plugin should not expect or assume that any other person in the world is wi...
i forgor that timezone strings exist
quaint sapphire
This plugin should not expect or assume that any other person in the world is wi...
Ain't no way I'd give Europe/Athens instead of Greece Athens
The former is the timezone, the latter is the actual city
solemn spoke
yeah ig random ass website with complicated idea is the best
turbid hatch
https://discord.coffee/ABXdTfR.png yeah it does
!!!!!
austere talon
the idea is that other people can give you their timezone
turbid hatch
just make a /command or something
austere talon
like your normie friend who doesn't use mods
quaint sapphire
That ain't the idea tho
quaint sapphire
Both me and manti did not envision that
austere talon
but your normie friend who doesn't use mods will not be able to give you a proper timezone because they're stupid so it's easiest to send them a link they click and immediately get the correct timezone
crude hearth
doesnt it get timezone of users automaticly
crude hearth
rusher will add nonlogin option to timezonedb website
turbid hatch
link makes sense
crude hearth
we use it
quaint sapphire
but your normie friend who doesn't use mods will not be able to give you a prope...
Or they could tell you the fucking city and you pick it from a list
austere talon
rusher will add nonlogin option to timezonedb website
yes just make Homepage show current timezone
turbid hatch
although having it in client is better too
solemn spoke
but your normie friend who doesn't use mods will not be able to give you a prope...
but how would you do it
crude hearth
@fiery jungle RUSHERRRRRR
austere talon
Or they could tell you the fucking city and you pick it from a list
Okay I live in [arbitrary city that isn't a valid IANA timezone]
not everyone lives in the capital city
solemn spoke
fetch random ass api to find the nearby cities near you lnfao
austere talon
if you ever ask a human for their timezone you're doing it wrong
make a website that tells you your timezone like I said or something similar
quaint sapphire
Okay I live in [arbitrary city that isn't a valid IANA timezone]
Who would give a city other than the capital city or a close by city?
solemn spoke
i mean, making them to open a website is also weird
crude hearth
timezonedb plugin will get other persons ip adress and get timezone string from that
quaint sapphire
I doubt people want to dox their location
austere talon
oh my god this discussion is stupid just implement whatever and if its shit we change it and if its good we keep it
I think what I suggested is a decent idea
crude hearth
yes rusher does most of the work
solemn spoke
tho very complicated
crude hearth
sounds good to me
quaint sapphire
Why are you suggesting stuff when manti has not fixed the original issues?
austere talon
but if you have a better idea (that isn't hardcoding a timezone list and making stupid users figure out timezones) then go for it
crude hearth
Why are you suggesting stuff when manti has not fixed the original issues?
what are the original issues
also even tho we are probably gonna use website to get timezones
I think it would be better to keep autocompleteselect
because copy paste would work on that
also
quaint sapphire
what are the original issues
Trying to get the timezones from intl and if it fails to fetch them from a site is one of them
crude hearth
Trying to get the timezones from intl and if it fails to fetch them from a site ...
thats literally only thing I need to do
quaint sapphire
Okay I live in [arbitrary city that isn't a valid IANA timezone]
That's why I like GMT offsets, but summer time savings are an issue
solemn spoke
but if you have a better idea (that isn't hardcoding a timezone list and making ...
i mean, i think opening a website just to do the work is over engineered. iirc the timezone polyfill support you to just do a UTC+3 in it
wait, is that a general IANA rule or smth?
quaint sapphire
Who cares about summer?
turbid hatch
also, its not really overengineered
solemn spoke
if you put the IANA timezone as UTC+3 for example, it is valid
crude hearth
https://gh.lewisakura.moe/timezone/
logger
I am started to think ven is right
we should just force other users to authorize
solemn spoke
https://gh.lewisakura.moe/timezone/
i thought the website would literally send back the timezone back to the one that sent the link. as like:
*user opens website that his friend gave*
*website sends the tz to the friend*
*the plugin would add to the list*
austere talon
https://discord.coffee/ABXdTfR.png yeah it does
it's this simple to get the users current timezone. Literally slap this on a site and tell anyone whose timezone you need to go there, super simple and they don't need to use brain
<h1>Timezone</h1>
<p id="balls"></p>
<button onclick="copyTz()">Copy Timezone</button>
<script>
const tz = Intl.DateTimeFormat().resolvedOptions().timeZone;
balls.textContent = "Your timezone is " + tz;
function copyTz() {
navigator.clipboard.writeText(tz);
}
</script>
oh lewi was faster based
quaint sapphire
i thought the website would literally send back the timezone back to the one tha...
Why not make it a request thing within the plugin if we went that route?
austere talon
it be working
turbid hatch
yis
quaint sapphire
Maybe the plugin could send a DM with the timezone?
turbid hatch
ill make it look pretty now
solemn spoke
okay the solution: literally don't solve it and make the user to manually add it, assuming the friend knows their IANA timezone, but if they don't, then the plugin would suggest a website where their friends could check which IANA tz they use
turbid hatch
cause im quite fond of it
austere talon
lewi doesn't know that any dom element with an id automatically becomes a js variable
turbid hatch
why does the plugin need to send a dm
austere talon
turbid hatch
lewi doesn't know that any dom element with an id automatically becomes a js var...
wait what
quaint sapphire
lewi doesn't know that any dom element with an id automatically becomes a js var...
?
solemn spoke
okay the solution: literally don't solve it and make the user to manually add it...
what about that? sounds a good solution for me
austere talon
<p id="lol"></p>
<script>
lol.textContent = "lol. lmao even"
</script>```so silly
austere talon
quaint sapphire
I've been programming in JavaScript for years now, I know a lot of things about JavaScript, but I never expected that nodes with IDs have auto variables
austere talon
ye it's more explicit and more typescript compatible
solemn spoke
what about that? sounds a good solution for me
or solution 2: force friend to sign in for random ass service
austere talon
but I like abusing obscure behaviour
crude hearth
it's this simple to get the users current timezone. Literally slap this on a sit...
yes thats good way to solve it but just ... I cant think asking someone to enter to website and paste the string into me so I can set their timezone
this is getting insane
austere talon
??
turbid hatch
what
the flow is so simple
give user link (https://gh.lewisakura.moe/timezone/)
user goes to link
user copies from here
user gives to you
quaint sapphire
ye it's more explicit and more typescript compatible
I now understand why it is not recommended to use the same id for multiple nodes
austere talon
you think asking a clueless person for their iana compliant timezone is less of a bother than making them click a website
austere talon
I now understand why it is not recommended to use the same id for multiple nodes
no that's just cause its technically undefined behaviour
crude hearth
you think asking a clueless person for their iana compliant timezone is less of ...
its a bother everyway whatever
solemn spoke
yes thats good way to solve it but just ... I cant think asking someone to enter...
that's understandable, considering that it will give where the person lives in some way, and the person will not know why you need it
turbid hatch
if you're asking someone for a timezone
you're probably already on a decent enough level with them
austere talon
the dom spec says only one element may have the same id so technically any browser may choose to optimise for that case and for instance make query selector all for id only return the first match (but I don't think any browser does that, they all work fine with duplicate ids)
quaint sapphire
you think asking a clueless person for their iana compliant timezone is less of ...
I don't know, nobody I know considers cities as timezones
GMT and UTC are what we would consider a timezone
turbid hatch
and either way the timezone is still quite generic, london is very far from where i live
solemn spoke
if you're asking someone for a timezone
they'll think of -3, not America/Sao_Paulo
austere talon
What's your tz database timezone name?
2:41!!!
solemn spoke
I don't know, nobody I know considers cities as timezones
GMT and UTC are what ...
that's the point
turbid hatch
2:41!!!
Whoops! Not quite Ven, that's an offset! Do you mind going here for me? https://gh.lewisakura.moe/timezone/
crude hearth
ok so I got another idea
solemn spoke
go ahead
quaint sapphire
Also it is weird that GMT+2 is Etc/GMT-2
like, what weed was iana smoking?
crude hearth
ok so I got another idea
so instead of forcing user to enter a website we ask them whats the time for them, then when they send timezone we will add their timezone via slash command which will figure out the utc timezone itself
but the problem with this is winter time
again
austere talon
not possible
crude hearth
not possible
why
turbid hatch
you are dealing with time
crude hearth
like I asked you, you answered 2:42
austere talon
just make Website there is no better user friendlier way
crude hearth
like I asked you, you answered 2:42
2:42 is gmt +1
turbid hatch
right i can make this website attractive as fuck
quaint sapphire
I'd like a clock and you pick the time of the other person and it shows possible timezones to pick from that are currently at that time
austere talon
like I asked you, you answered 2:42
you have no way of knowing what timezone I'm in though
turbid hatch
you're now back to the issue we had before that you complained about
austere talon
crude hearth
you have no way of knowing what timezone I'm in though
yeah thats why I said winter time etc is a problem
you only know offset
turbid hatch
just give them the god damn website and ask them
austere talon
yall rn are like "I don't like this super simple solution you already implemented, here's [overcomplicated less user friendly solution]"
just stop and use the website lewi made, maybe put it on timezonedb domain
quaint sapphire
I'd like a clock and you pick the time of the other person and it shows possible...
You could find the iana tz city that way, by just asking for the current time
austere talon
you could also wait for them to send a screenshot that contains their time then guess it based on that
now you don't even need to ask them!!
tame escarp
why dont yall just not make the plugin because this argument has been going on forever
crude hearth
okay so I am thinking
quaint sapphire
you could also wait for them to send a screenshot that contains their time then ...
austere talon
wonderful idea (not), just use the website 
crude hearth
if we are gonna go with website way, we should add a slash command that will send website link, and instructions about what to do
solemn spoke
like
cant we literally just not deal with it and just give a warning that it is better to use a IANA timezone offset if they use a UTC offset?
crude hearth
if we are gonna go with website way, we should add a slash command that will sen...
then at least we dont have to look to website from somewhere
turbid hatch
if we are gonna go with website way, we should add a slash command that will sen...
right okay now we're on the right line of thinking
quaint sapphire
Or even an hour:minute offset from the local time
austere talon
oh my god you all need to shut up now this is going nowhere
you already have a finished solution thanks to lewi, either use that or implement your own better solution but shush now
austere talon
wdym? @spark cove
it'll show up in any build as long as you have source maps enabled
spark cove
how would you enable sourcemaps in a prod build as a user
austere talon
I don't understand that question
source maps are part of the release and downloaded by the installer & updater
enable source maps in devtools and they'll work
spark cove
source maps are part of the release and downloaded by the installer & updater
ah i didnt know they were already downloaded
makes sense then
spark cove
no way ur downloading the extension/userscript in a desktop installation
austere talon
Well ig browser and userscript LEGAL.txt are essentially useless
but I kept them there for blah blah legal reasons
austere talon
no way ur downloading the extension/userscript in a desktop installation
haha of course not
spark cove
yea i figured that wasnt true
i realize u meant that they're used somewhere and u can pretty much deduce where they're used
austere talon
desktop downloads renderer, preload and patcher js, js.map and LEGAL.txt, browser.js and css are used exclusively by armcord afaik but technically anyone can use them they just need to be ran in the renderer
user.js is well the userscript
spark cove
yop
austere talon
austere talon
i realize u meant that they're used *somewhere* and u can pretty much deduce whe...
haha yeah
I meant all of them are used in some way or another
fiery jungle
<@295190422244950017> RUSHERRRRRR
this conversation is so stupid
crude hearth
this conversation is so stupid
did you read entire conversation
fiery jungle
no
fiery jungle
yeah
maiden fog
if you ever ask a human for their timezone you're doing it wrong
i wish timezones were classified based on near atomic clock precision on a given latitude and longitude 
might be horror for privacy guh
cunning bobcat
No offense to manti
full offense to manti
hi manti
wait you're not manti
fucking hell
rare shale
im your husband
cunning bobcat
lies
rare shale
we had hot gay sex
cunning bobcat
False
woeful sable
crude hearth
full offense to manti
NOP
Guys civilization
Be civil
Dont go to bottom of human evolution tree
cunning bobcat
this message is offensive to any manti in the area
crude hearth
@austere talon I will ask you one question
now you are aganist autocomplete and say people should just copy paste timezones
but in your situation if people use the web browser which has (maybe) newer version and gives some different string
how would discord client on old version work
crude hearth
<@287555395151593473> it uses the system timezones
oh
I thought they were hardcoded
turbid hatch
nop
it pulls the available timezones from your OS' tzdb
so its the same regardless of version
also i saw your Etc/ request, those arent part of the tzdb
hence why they werent in the original gist
i could add them but mileage may vary
crude hearth
love
crude hearth
also i saw your Etc/ request, those arent part of the tzdb
it doesnt has to be in timezonedb
people will use this for local timezones
also ven is aganist getting hardcoded timezones
turbid hatch
shrug
crude hearth
horror
charred monolithBOT
crude hearth
moment also has method to get all timezones
but it requires moment-timezones lib 
I dont think discord has
oh god this is making me insane
we hate Intl for not having this option in old chromium
because I wanna add autocomplete I thought maybe it would make sense to get list of timezones from somewhere updated
BUT I DONT KNOW ANYMORE
http://worldtimeapi.org/api/timezone like this website
solemn spoke
I dont think discord has
it doesn't
umbral hedge
Bad Apples
One
austere talon
oh yeah you asked whether you can merge
I didn't look but if it's all good then yeah probably
what's staging banner
limber skiff
limber skiff
I didn't look but if it's all good then yeah probably
I did some lookbehinds and lookaheads studies to improve performance lol
austere talon
why did you rewrite so many SHC patches
austere talon
I did some lookbehinds and lookaheads studies to improve performance lol
I'm surprised those improve performance and don't worsen it
austere talon
what's the purpose of this?
limber skiff
nothing. just visual
limber skiff
I'm surprised those improve performance and don't worsen it
they were making it a bit worse, now they make it better
austere talon
I feel like this is a feature less intelligent people will enable then ask for support with hiding the banner
austere talon
they were making it a bit worse, now they make it better
wdym by that
austere talon
nothing. just visual
then why did u add a patch for that, why would anyone want that little banner
limber skiff
lets say I have this regex
/(?<=function (?<func>\i){.{0,900}.MESSAGES.BLABLA.+?}});/
this regex would first match the lookbehind, so it would try to match every function \i and look for .MESSAGES.BLABLA, which is not good
now I made it like this
/.MESSAGES.BLABLA.+?}};(?<=function (?<func>\i).+?)/
It's gonna match .MESSAGES.BLABLA first, which is not common, and then lookbehind for the first function \i to get the name and save to a group
limber skiff
then why did u add a patch for that, why would anyone want that little banner <:...
someone asked, it's just something additional
limber skiff
now I made it like this
```
/.MESSAGES.BLABLA.+?}};(?<=function (?<func>\i).+?)...
By doing this I was also able to remove a lot of fixed size quantifiers which should make plugins break less
limber skiff
actually
I will make pronouns db not enable the experiment but still show the component
then I will merge
turbid hatch
now I made it like this
```
/.MESSAGES.BLABLA.+?}};(?<=function (?<func>\i).+?)...
and what's the performance improvement?
cause if its not a lot you've just made loads of patches massively unreadable for no reason xd
also you seem to have a misunderstanding of how those operators work so i hope you didnt merge it
limber skiff
how so
turbid hatch
well every example ive seen has shown the lookbehind come before
never after
and i can't get it to work in regex101
so i have no idea what your patches are matching
your own example doesnt compile either ๐ฅด
im just trying to understand how this actually is meant to work and if its speeding up anything or if its just making the regexes harder to maintain, or if its even working at all
but the more i look into it and the more i test, the more confused i get
limber skiff
I explained it in #๐งฉ-plugin-development
turbid hatch
yes
i tried going through your explanation and even tried one of your patches in regex101 but it didnt match anything lol
limber skiff
select the ecma script flavor
sour drum
now I made it like this
```
/.MESSAGES.BLABLA.+?}};(?<=function (?<func>\i).+?)...
Wait that's actually really smart. I'm going to update my regex to use lookbehinds
cunning bobcat
wtf is a look behind
sour drum
wtf is a look behind
" python " 
sour drum
wtf is a look behind
I'm still looking into it a bit more
cunning bobcat
YOOO A ONESHOT PLAYER
charred monolithBOT
Just to make sure, did you try this?
austere talon
you need to include the match in the lookbehind as well
cunning bobcat
mfw github weebhooks bad
turbid hatch
you need to include the match in the lookbehind as well
oh i see
hm
thats still really confusing xd
undone delta
charred monolithBOT
Description: Searches Wikipedia for your requested query. (/wikisearch)
(If you have any better description ideas please comment I beg of you)
quick ibex
Image for any discordian lurkers
austere talon
how do u use that @limber skiff
austere talon
ah
i got confused cause you also edited context menu api
so i thought it used context menu
and was confused
limber skiff
I fixed it lol
limber skiff
jeez I completely forgot about startsWith
austere talon
lol
i thought for like 10 seconds "is there any specific reason they might have done slice instead of startsWith??"
i think startsWith might just do a slice (without creating a new string) and comparison
limber skiff
no lol I just forgot about it
I was like include wouldnt do what I want...
slice then.
austere talon
how did that even happen
do u always copy paste the boilerplate from some other plugin or template or
limber skiff
what happened
austere talon
2022
limber skiff
It is needed
enabled doesnt update without it
idk kinda strange but it worked with it lol
turbid hatch
wouldnt it be more reliable to set the flags on the message rather than appending <@&1073655264923881602> to the content? or is it fine in this case
limber skiff
can't
limber skiff
it's too early lol
turbid hatch
mkay
austere talon
maybe this could be a @util/react thing
limber skiff
eh
I think it's fine using useState there for now
we do it like that everywhere else
austere talon
function useBoolean(initialValue: boolean) {
return React.useReducer(x => !x, initialValue);
}
// usage
const [value, toggle] = useBoolean(false);
<Switch value={value} onChange={toggle} />
austere talon
wtf is SettingsStore
limber skiff
I didn't know how to call it lol
austere talon
i feel like that should be a webpack method instead
limber skiff
it requires patches to work with it
discord has basically this
each is an object which contains functions to manipulate that setting
get its value, update it, etc
to find it I need to patch to add additional information to the object
else they are all the same
should I update these
austere talon
no
updating years in license headers is meaningless and ruins git history and is also misleading if you don't change anything else
but new files should use 2023
woeful sable
we will remove years from license headers
austere talon
years are fine if they just correspond to the year the file was created
quick ibex
You love 33 open prs
austere talon
ye
like half of them aren't even done / mergable
the other half I already reviewed mostly, gotta check again whether feedback was addressed
btw on the topic of feedback if I left some on ur pr and you fixed everything feel free to ping me for rereview
cause its not easy to keep track
quick ibex
Can you review my pr? Or is it too messy
turbid hatch
ill review it
await fetch(`https://en.wikipedia.org/w/api.php?action=query&format=json&prop=info%7Cdescription%7Cimages%7Cimageinfo%7Cpageimages&list=&meta=&indexpageids=1&pageids=${firstres.pageid}&formatversion=2&origin=*`)
.then(data => data.json())
.then(data => alt_data = data.query.pages[0] || null);
thank you dedent
quick ibex
@turbid hatch thank you, been a while since I coded so I forgor any and all kind of error handling
turbid hatch
^^
charred monolithBOT
quick ibex
and also did some cleaning up
charred monolithBOT
grave mangoBOT
https://github.com/Vendicated/Vencord/blob/main/src/plugins/invisibleChat/index....
**index.tsx: **Lines 72-79
/*
many themes set "> button" to display: none, as the gift button is
the only directly descending button (all the other elements are divs.)
Thus, wrap in a div here to avoid getting hidden by that.
flex is for some reason necessary as otherwise the button goes flying off
*/
<div style={{ display: "flex" }}>
<Button
limber skiff
charred monolithBOT
I'd like to suggest a plugin similar to Voice Chat Utilities. I personally really like this plugin, as it allows you to move, mute or deafen, and disconnect all people from a voice channel.
charred monolithBOT
lime stone
verbal pumice
:ablobwave:
charred monolithBOT
