#topgg-open-source

1 messages · Page 1 of 1 (latest)

woven granite
steady forum
woven granite
#

I cant test it, I'm not a Java dev.
But everyone I've asked to test it says it works. So it should be okay.

#development message

steady forum
steady forum
tiny shard
dusky shale
tiny shard
dusky shale
tiny shard
dusky shale
#

Do you have access to the repo? It seems that only Xetera has been committing changes

dusky shale
dusky shale
wintry marten
#

I don't have write access to anything anymore

#

maybe someone from the team could help u with it though

dusky shale
#

Oh damn, I'm not sure who would have write access, maybe @steady forum or @deft merlin? (Have a look at the PR)

tiny shard
#

yeah i can only merge on node-sdk, not sure who’s really in charge anymore lol

wary stratus
#

Understandably ngl

#

Xiuh is on maternity leave, so Mac is the only one to reach out to

tiny shard
#

@dusky shale you’re planning on making changes to the node sdk right?

dusky shale
#

Yes, though I gotta fix this dependency conflict first

tiny shard
#

gotcha

#

ok i was gonna update topgg-autoposter soon so i’ll hold off until this

steady forum
#

I've passed on the request. Neither Xiuh nor myself have access to this repo. I have access to some open source repos but not all of them darkAYA @tiny shard @dusky shale

tiny shard
#

gotcha

#

yeah, it would be neat if me or another oss contributor could get review access to eslint-config, i already have node-sdk but considering itself is a dependent and can cause problems like this one, it could reduce headaches

steady forum
#

Yeah agreed. If I can get admin on this I'll add you darkLike

tiny shard
#

(tho not sure how the bureaucracy of top.gg open source looks like nowadays lol)

tiny shard
steady forum
tiny shard
#

IC everything is under medal now gotcha

#

sounds good, keep us updated catyes

steady forum
#

Yep will do! darkLike Thanks for the PR darkPray

steady forum
#

@tiny shard @dusky shale it's been merged darkPray

dusky shale
#

@tiny shard I believe you can do that since you're a collaborator to @top-gg/eslint-config npm package

steady forum
#

Ok if berry can't do it ping me and let me know

tiny shard
#

The github workflow is supposed to do it, but it’s set to the wrong branch ohno

#

I do believe I’m able to push straight to npm tho

tiny shard
#

Just gonna manually publish so we can move forward finally

#

@dusky shale @top-gg/eslint-config@0.0.3 is available in npm now

#

Let's remember to fix the workflow if we need to make changes again

steady forum
#

Passed it on

tiny shard
#

Voltrex, this whole thing reminded me actually, make sure to manually bump the version in package.json in your PR to node-sdk, there's no autoversioning

dusky shale
tiny shard
#

just update it in the file it’s fine

#

it’s not an automated task it’s on release anyways

dusky shale
#

Alright, will do so

steady forum
tiny shard
#

thanks!

dusky shale
#

Nice

#

Though note that you'll have to fix the Netlify deployments as it's broken, check the deployment logs for info

wary stratus
dusky shale
tiny shard
#

@steady forum cc @wintry marten was the Top.gg (topgg-node-sdk-docs) netlify ever transferred over?

tiny shard
dusky shale
# tiny shard Added some comments

Asking here so I don't clog up the replies there, what of Undici's request() method is better in your opinion? The fetch() is generally faster and more secure

#

And considering that fetch() is a standard, it's better IMO

tiny shard
#

I think the request standard just looks like nicer, abstracting to a body and therefore the ability to deconstruct other properties out of the request() return would be a nice replacement of what's there right now (atleast without a ridiculous text()/json() variable)
There's no real security or speed difference atleast not in our application & comparing Undici's request & fetch method, I do agree both are better than node-fetch tho

#

I also enjoy undici's stricter typings

dusky shale
tiny shard
#

to do this with fetch it would look like

const { json, text, status ... } = await fetch(...)

await json()
await text()

Which I do not like at all, the variables are very indescript
with request it's

const { body, statusCode ... } = await request(...)

await body.json()
await body.text()

As for typings undici has stricter typings with methods and headers as the most obvious ones

#

Also, big one, node 14 (which I guess we're still trying to target) has no support for Undici#fetch

dusky shale
#

Oh, yeah I see your point on the names of those properties though what I'm worried about here is mainly that the request() method of Undici is pretty basic and doesn't handle most of the things (request() was only made to be used in environments where not much operations can be done), and yeah fetch() does drop support for Node.js v14 though not much people use it anyway

#

But if you still want me to use request(), sure

tiny shard
#

I can see the argument if our application was more complex, but really we're not doing anything crazy, the code style, support and typings are enough to convince me

dusky shale
#

Alright, I'll use the request() method then, though something in your usage of the <span> tag confuses me, how come that docs website you mentioned turns Top.gg into a link but it doesn't do that for Topgg.Api?

tiny shard
#

I'm not sure honestly, rtfd is a mess, definitely going to move away from it regardless but we still autobuild to it and it still has some traffic

dusky shale
#

Actual mess, good luck migrating away from it

tiny shard
#

you can see the old theme here too btw ^

dusky shale
#

I saw, though the new looks way better IMO, it also has a light and dark theme and prefers it based on the OS default

tiny shard
#

from gh

I mean we can do it, but I really like how the minimal theme looked, module selectors, and larger settings options seem really unnecessary for a project with two classes and a couple of types.

#

I also like how you can navigate the entire project w/o having to go back up to the module scope

#

Wouldn't be preferable in larger projects but in this one it's nice

dusky shale
#

I don't know if there's a theme similar to the minimal one ¯_(ツ)_/¯

tiny shard
#

Not sure either, could you go and look for one? If not we can go to the default or consider going to an older version of typedoc honestly. Let me know

dusky shale
tiny shard
#

Hm

#

Can you look and see if there’s a way to remove the module selector? That’d make me happier

dusky shale
#

What are you referring to by the module selector?

tiny shard
#

Also maybe if there's a way to not have the other classes/types collapse when selecting one of them

tiny shard
#

Compared to the current ^^

#

We don’t really have a choice but to switch, but take a look around and see if you can find any options that’ll make it a little bit simpler

dusky shale
tiny shard
#

Alright, is what it is ig

dusky shale
#

I'm saying we should just use fetch(), it drops a lot of burden and already handles everything we need

#

The only drawback is that we would need to drop support for Node.js v14, as not much people use it anymore anyway

tiny shard
#

Honestly that’s the dealbreaker for me. There are lots of people who still use 14 for discord bots

#

Also regarding headers we don’t take any manual headers in

dusky shale
#

The main user base of the node-sdk is discord.js users anyway, and considering that discord.js requires Node.js v16...

tiny shard
#

There are lots of people who use outdated versions of Discord libraries because they don’t care for slash commands. It wouldn’t be such a big deal if 14 wasn’t so like, not long ago

dusky shale
#

v13 also requires Node.js v16, which is the majority of their users

tiny shard
#

Sure but we just throw people out because we want to use a fancy (imo less fancy) request library?

#

Let’s move forward with request, if it becomes too big of a problem we can always drop it in the future

dusky shale
#

Do you want to take the extra burden of handling the things request() doesn't handle itself just for the minority of the users? It has also been reported many times that request() causes unexpected issues that gets fixed multiple times repeatedly when it's not the case for fetch()

But it's your choice, I see why you want to preserve backwards compatibility but it's not really worth it tbh

tiny shard
#

If it becomes that big of a deal we can move

dusky shale
#

Alright, or we can use request() until Node.js v14 reaches EoL stage which'll be soon

steady forum
tiny shard
#

Sounds good, thanks!

tiny shard
#

@steady forum poke

steady forum
tiny shard
steady forum
#

I don't know if it is. I was just asking if there were any other tasks you wanted me to poke about.

tiny shard
#

Only looking for netlify, once that build passes I can do merges and releases

steady forum
#

Ok. I'll poke about it now.

tiny shard
#

@dusky shale @top-gg/sdk 3.1.4 🎉

#

Thanks for your contributions!

steady forum
#

GGs! dviperPartying Thank you so much for fixing this @tiny shard @dusky shale peppapog

dusky shale
tiny shard
dusky shale
tiny shard
#

just need to update everything

drowsy thistle
dusky shale
tiny shard
#

when was the discriminator change made? @drowsy thistle

#

still thinking it'd be better to @deprecated

drowsy thistle
#

about two years ago

dusky shale
#

Honestly it would be better to just remove it now instead of still marking it as deprecated, it's been deprecated for 2 years anyway

drowsy thistle
tiny shard
dusky shale
tiny shard
dusky shale
#

Correct, although many projects tend to publish a release with the breaking change (while bumping the SemVer major version of course) after a long time, which 2 years already is a very long time, but I understand if you're planning to do it when you've got another breaking change more important than that

drowsy thistle
#

to be honest

#

why did they even remove the discriminator property in the first place

#

like "hold on lemme quietly break several people's bots rq"

cunning locust
#

netlify moment sadge

image.png
drowsy thistle
cunning locust
#

lemme check - i've not changed that

#

can't test on other bots

#

wait yes i can i am stupid

#

yep its a number not a string, will adjust now

image.png
tiny shard
#

@drowsy thistle ok yeah let’s make it @deprecated instead please.

#

Also, please bump the package version

drowsy thistle
tiny shard
steady forum
#

Wasn't he pinged in the PR already?

cunning locust
steady forum
tiny shard
#

thanks

drowsy thistle
tiny shard
#

@drowsy thistle released 3.1.5, thanks!

drowsy thistle
#

no worries! glad to help! kittyLove

drowsy thistle
#

@dusky shale how's this poggythumbsup ```rs
#[macro_use]
extern crate rocket;

use topgg::rocket::IncomingVote;

#[post("/dblwebhook", format = "json", data = "<vote>")]
fn dbl_webhook(vote: IncomingVote) {
match vote.authenticate(env!("TOPGG_WEBHOOK_TOKEN")) {
Some(vote) => println!("{:?}", vote),
_ => // 401
};
}

dusky shale
drowsy thistle
tiny shard
steady forum
#

Hi! Sorry no update on this right now. We're kinda swamped on the Medal side at the moment so he hasn't had a chance to check it out. I'll poke him about it! mayaPray

tiny shard
tiny shard
drowsy thistle
tiny shard
drowsy thistle
#

it's just an example

tiny shard
#

oh wait

#

ok yeah mb

drowsy thistle
#

@steady forum sorry about the amount of commits by the way - i realised that your response was taking a bit too long so i decided to make changes along the way

steady forum
#

No problem! mayaHappy I noticed you were still adding to it so I wanted to wait until you were done. I've reverted it to a draft. Please open it again once you've totally finished.

drowsy thistle
#

alright! i thought you were too busy with medal onesieKEKW

steady forum
#

Also that

dusky shale
tiny shard
#
  • force pushing is really annoying as a maintainer
drowsy thistle
tiny shard
#

dw about it

#

force pushing isnt necessary nor is it a good idea

dusky shale
dusky shale
# tiny shard + force pushing is really annoying as a maintainer

I disagree, most projects prefer force pushing as it keeps the activity timeline clean and doesn't make it so that the maintainers have to keep track of all the changes in every commit, besides, force pushing is also better with tooling that operates on commits as working with as few commits as possible makes it easier to manage

tiny shard
#

just squash

#

its why its there

dusky shale
#

The only times where force pushing is not as good is when there are tons of changes (like thousands), at that point individual commits help

tiny shard
#
  • you cant see changes over time without tracking previous force pushes, meaning it's more difficult to see what changes you actually made
  • it's good to see reactive changes to comment reviews, especially with a commit message the explains it
  • you're now limited into a single commit which means you'll often be forced to use a commit message that isn't descriptive
  • being able to track even just a couple lines of code to a commit that also likely has a more descriptive message + a separate description, if you use git correctly mmLol
  • squashing is not difficult, and squashing pull requests are good in nature as you can track back to specific pr numbers, more commits just incentivizes squashing
  • force pushing is risky, maybe you know how to but saying other people should is a really good way to mess someone up because they just overwrote all of their code, hence why its a force
#

its like literally in the name

#

as a maintainer it's obnoxious to work with pull requests of people who just force push over and over again, you're not the only one, and everytime it sucks

#

when you're making 10-20 line or single file changes it doesnt really matter, but when you're doing things like what null is doing it makes a big difference

dusky shale
# tiny shard - you cant see changes over time without tracking previous force pushes, meaning...
  1. Seeing changes over time is rarely ever needed because the latest changes are the ones that matter, and besides Git hosts generally keep the commits overwritten so you can check those as well
  2. This becomes a burden and unnecessary when there can be a lot of reviews, writing a descriptive message for literally every review addressed/commit pushed just increases effort without providing much use
  3. Generally PRs are supposed to focus on a change they make (especially the one written in the title), not to make multiple changes that can diverge from each other, which makes it hard to track or manage, this can be seen in many places where such things are frowned upon and tell you to open a separate PR instead
  4. Point 2 and 3
  5. Squashing is useful yes, but isn't supposed to be used all the time, again it's more preferred to make individual commits and squash on very large changes
  6. It indeed is risky, but also helpful and useful especially for tooling and whatnot, users should learn the overall concept and workings of force pushing so they wouldn't make such mistakes in the future where force pushes are required
tiny shard
#

1 is just not true?

#

if you have a massive PR like nulls its VERY useful to see the individual commit rather than trying to decipher the entire PR to figure out what changed

#

if what you said is true than why dont we just have 1 commit repositories

#

its extreme but is also the same point

dusky shale
#

In our teams of reviewing that I've talked to (more than thousands of people), rarely any of them have ever needed to track changes over time in the activity timeline, they only keep track of the very latest changes and review it, and again the Git hosts keep the overwritten commits as read-only for viewing so you can still see the changes in a way

tiny shard
#

then they're just not dealing with large prs? idk cuz this is an invaluable feature lol

#

also github isnt always the tool

dusky shale
#

We deal with very large changes all the time, and for the second thing you said, I know and that's why I said "Git hosts"

#

Generally all of them keep it as read-only

tiny shard
#

sure but having to pull them in and diff them is still obnoxious?

dusky shale
#

You don't have to pull them in though, the interfaces usually show them as detached commits that no longer belongs to the repository as it was force pushed, but the changes are still there

tiny shard
#

right but now you have to get the 2 sha's and compare them rather than just look at the commit

dusky shale
#

Pretty much every Git host I've used (GitHub, GitLab, Git Bucket, and others) show the two commit SHAs that were force pushed in the activity timeline

#

Even Phabricator does it lul

tiny shard
#

i often do diffs in my code editor for larger prs

#

i work with very very large changes daily at my job

#

and some commits come in with 100s of lines a day in difference and i just want to see that

#

because im already confident about the changes made before

dusky shale
#

Huh, I've never seen anybody else do that, everyone I've talked to just use the interface of the Git host

tiny shard
#

again if it's just a couple lines or a couple files its fine but refactors like this is just terrible to navigate especially if you've already gone in and approved everything before it

dusky shale
#

I already said that individual commits are better for large changes, was Null's changes large? I didn't actually check

tiny shard
dusky shale
#

Oh, then that's understandable, I didn't know it was thousand lines of changes

#

Should've just checked DogKek

cunning locust
drowsy thistle
cunning locust
#

yeah

dusky shale
#

They're revoked as soon as they're sent anywhere on GitHub

#

Private repositories being the exception of course

cunning locust
#

oh awesome, did not know that

wary stratus
cunning locust
#

doesn't look like it

cunning locust
#

just gonna drop this here

dusky shale
#

Also I would rather call it "section" instead of "location"

cunning locust
#

Alrighty

#

and i'll fix that now

drowsy thistle
cunning locust
drowsy thistle
#

(btw read my follow-up comment)

cunning locust
#

just seen, the Libraries.jsx file is unused

#

just removed it, apart from that it all looks good and I'm just fixing what Voltrex said

drowsy thistle
#

nice

dusky shale
cunning locust
#

oh? it does on my screen

dusky shale
#

It doesn't on mobile probably

cunning locust
#

tested on my phone too and it worked sussy

drowsy thistle
dusky shale
cunning locust
#

ah yeah i should make that clearer

tiny shard
#

This looks awesome. And I hate to be cynical, but I worry about the implications of creating a website from scratch meaning this repository would be in charge of maintaining, especially with how slow the repository is with approvals, there’s an advantage to having the site and theme made by another dedicated library.

cunning locust
#

Isn't that basically how it works at the moment?

#

I think for faster approvals, there just needs to be someone else who is trusted to have access to merging, it looks like only Veld is a reviewer

tiny shard
cunning locust
#

ah

tiny shard
#

it basically handles everything for us

tiny shard
cunning locust
#

True, I doubt this would need much maintenance however, bearing in mind like for the current repo, hardly and modifications are needed and it would work without any frequent updates/maintaining needed

tiny shard
#

In a perfect world sure, but inherent to managing a site outside of a library that does it for you puts the burden of maintenance, tech debt, updates, dependencies, security, onto the repository

#

you can say it’ll never need maintenance but this is rarely ever true

#

just sort of inherent to software

cunning locust
#

True

drowsy thistle
#

@cunning locust have you merged my pr

cunning locust
#

Working on that now

#

Done

drowsy thistle
#

nope

cunning locust
#

its just deployed

#

oh i manually moved to my main PR

#

the one on top-gg/docs

drowsy thistle
#

i added several commits

#

in my pr

cunning locust
#

yes i added those all

drowsy thistle
#

good

cunning locust
#

including the bits where you reworded from an to a

#

added this at the bottom

image.png
drowsy thistle
cunning locust
#

now we patiently wait

#

@steady forum i didn't request it from veld, that was auto from Github because he is the only reviewer 😭

drowsy thistle
cunning locust
#

wut

drowsy thistle
#

bottom

cunning locust
#

oh

#

not intentional 🙏

#

also whats the white going over the screen because thats not showing on mine

drowsy thistle
#

idk mobile fuckery

cunning locust
#

ofc

#

if needed i can just remove the purple grid

#

i might do that

#

mb, fixed and pushed

steady forum
cunning locust
#

github confuses me sometimes

steady forum
#

fr

tiny shard
steady forum
#

stinky

tiny shard
#

hes the only valid codeowner lol

cunning locust
#

I moved CODEOWNERS into /credits so people that worked on libraries get their rightful credits 🙏

steady forum
#

Do you want the icon with the "Top.gg" btw? This looks weird.

Screenshot_2023-04-20_at_19.13.56.png
tiny shard
#

it has to be in the root directory

cunning locust
cunning locust
steady forum
#

you're going to need a light and dark theme version of it

cunning locust
#

alrighty

tiny shard
steady forum
cunning locust
#

tyty

steady forum
cunning locust
#

I opened a PR because the current one is

  1. outdated
  2. a literal copy of another site's docs
#

plus this new one follows Top.gg's branding, whilst the other does not really

tiny shard
#

Also it's not a copy it's a library, the theme is used for many sites

#

I think the site is beautiful, I do think it would be a good improvement, it just brings up maintainability issues is all. What happens when you or whoever else was working on it doesn't want to be involved anymore? etc.

cunning locust
#

such as https://docs.top.gg/api/bot/

the layout is quite overwhemling:

  • everything is squished together, no seperators
  • feels all over the place, endpoints aren't together (votes endpoints should be together, stats endpoints should be together

imo, it just needed a redesign mostly to be more user friendly and to make it more consistent with both the content and the branding

cunning locust
#

updates are easy to be made, and locally hosting to work on it takes like 2 commands

dusky shale
cunning locust
#

and that yes

#

apollo is not a docs library, the docs came from apollo on github

dusky shale
#

Years later, social card never got fixed

image.png
tiny shard
tiny shard
#

And again, you can say there will never be maintenance, but this just isn't true, like I said before

tiny shard
dusky shale
#

I'm just quoting what the author said

tiny shard
#

the apollo docs is made to be able to host other content

#

same point still stands, we don't need to maintain it other people do

dusky shale
#

Not saying it's copied or not, they said it's copied themselves now I don't know krShrug

tiny shard
#

There is no copying it's a theme package

#

if you consider npm copying then i guess sure

dusky shale
#

Ask that Xetera

#

I am simply, once again, quoting

#

And while I do agree it's more maintenance needed which isn't necessary, bun said they could see themselves maintaining it krShrug

tiny shard
#

ok so you just ignored what i said

#

whatever

#

it is a library

#

other people maintain it

#

thats the bottom line

dusky shale
#

My point what about the copying part, never talked about maintenance nor it being a library in the first place

#

And about that, I totally agree as mentioned above.

steady forum
#

I agree with Berry on this. We won't be moving to something else. If there are issues with the content of the current docs feel free to open a PR to fix those. It does look better, but at the sacrifice of more maintenance problems, something we cannot afford to have. It's worked very well so far, I don't see a need to change what we're using already.

drowsy thistle
#

what about the other Pull Requests?

cunning locust
cunning locust
tiny shard
#

But in general I think there needs to be a restructuring of repo controls

#

Need to move out old maintainers and fix the CODEOWNERS issues

#

Maybe having 2-3 library developers approve PRs (outside of library contents) can be a merge? But I also understand not wanting to have volunteers have too much control

cunning locust
#

I think if a few trusted people were given the ability to approve PRs, and require 2-3 to approve before merging, that would be effective

#

i'm not a lib dev but i would happily help 👼

#

i did start a lib but sadly 2 days too late because top.gg started to make an official one just before i made mine with burbur 😭

drowsy thistle
#

tbh everything died after Xetera and Veld left

cunning locust
drowsy thistle
#

ik they're busy and all that but it's better than constantly waiting for Mac

cunning locust
#

yes but moderators are not that active either, community is faster and having multiple review is safer

drowsy thistle
#

again you can't really trust the community

cunning locust
#

yes but it would not be random people getting access

tiny shard
faint fern
tiny shard
#

thing with mods is that whoever would be dealing with it would have to involve themselves in our OSS space to actually understand what’s going on, else they’d just be hitting approve all the time without being able to verify the changes. So unless a mod stepped up to it it’d end up just being a lib dev or someone else asking a mod to approve and they just do it lol

#

though I do think a mod is definitely a considerable solution, they have more trust

woven granite
#

We do have a fair number of developers on the volunteer team to help out.

drowsy thistle
cunning locust
tiny shard
drowsy thistle
#

especially the docs

drowsy thistle
drowsy thistle
#

soooooo uhh will it be merged?

#

it's Tuesday

wary stratus
#

Its Monday*

drowsy thistle
#

almost Tuesday

drowsy thistle
#

@steady forum hey mac! sorry for bothering you, but can you merge this small pull request real quick? it's just a single line being changed, i swear it's the last one for this release! iara_pray https://github.com/top-gg/rust-sdk/pull/5

GitHub
fix: cargo publish error by null8626 · Pull Request #5 · top-gg/rus...

Hey Mac!
Sorry about this pull request - there was a slight discrepancy in the Cargo.toml file preventing it from being published to crates.io (apparently the period in top.gg is not allowed as a k...

drowsy thistle
#

sorry if i bother you with these pings btw iara_ded_lost

drowsy thistle
dusky shale
#

@steady forum is there a reason Null isn't given collaborator role in the repository of the SDK they maintain? It honestly feels better to have them be a collaborator instead of needing reviews from the Top.gg team all the time, since Null is the one maintaining the SDK anyway

#

Just like how Berry is a collaborator of Node SDK, Null should be collaborator of Rust SDK 🤔

drowsy thistle
#

though depends on what he meant by few

drowsy thistle
#

though it's a bit infuriating for me to constantly wait for them to review my pull request whenever i make changes

kindred maple
#

One message removed from a suspended account.

drowsy thistle
drowsy thistle
#

so uhhh will it be merged?

kindred maple
#

One message removed from a suspended account.

drowsy thistle
#

as expected

kindred maple
#

One message removed from a suspended account.

drowsy thistle
#

breastie

drowsy thistle
#

it's finally merged!!!

drowsy thistle
#

an update that fixes bugs!

wide crane
drowsy thistle
kindred maple
#

One message removed from a suspended account.

drowsy thistle
#

shut

high depot
kindred maple
#

One message removed from a suspended account.

deft merlin
#

perms are fucked up im fixing it

kindred maple
#

One message removed from a suspended account.

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
#

I'll check and see if we can get everyone's permissions sorted out, probably sometime next week

drowsy thistle
#

i have been waiting for like over two months now mimu_thisisfine

cunning locust
drowsy thistle
#

there are already 2 issues and 1 pull request in the repo and i can't do anything about it if i'm not even a collaborator

deft merlin
#

I'll see what I can do nod

drowsy thistle
#

thank you

kindred maple
#

One message removed from a suspended account.

drowsy thistle
#

this

kindred maple
#

One message removed from a suspended account.

deft merlin
#

heh ill let you know when i have an update, i promise

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

drowsy thistle
#

rare faithie W

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

drowsy thistle
high depot
kindred maple
drowsy thistle
#

this

kindred maple
#

One message removed from a suspended account.

deft merlin
#

I will be working on getting correct people access to the correct things here this week nod

drowsy thistle
drowsy thistle
#

soo erm

#

a week has passed

deft merlin
#

Yes, I just received admin on the repos last friday and I'm meeting with our external development team leader today to determine how we want to move forward with assigning maintainers nod

#

so expect news this week

#

You should know things move a bit slowly here onionpray but we are working on it

drowsy thistle
deft merlin
#

mimu_love_heart thank you for being patient. i know its been a bit of a bust recently but we're making changes to hopefully be able to better maintain things going forward

kindred maple
#

One message removed from a suspended account.

kindred maple
drowsy thistle
deft merlin
#

Hey everyone! I just went through and removed everyones access from our repositories whos names/githubs we don't recognize. If you don't have access to something that you should, please DM me!

We're going to be working on getting current active developers correct permissions to the correct repositories this week! Keep an eye out for a DM nod

deft merlin
drowsy thistle
cunning locust
#

replied to some of the open node-sdk issues

tiny shard
#

tracking the 504 fix

#

forgot to run the github action

tiny shard
woven granite
#

Hey, I'm the external development taskforce leader. I'll be reaching out to all the library developers today.
Mostly just want to touch base with everyone. ^-^

drowsy thistle
#

and since when was that a thing

deft merlin
#

Just recently we implemented taskforces within the volunteer team to have more people onboard and available to keep things on track

#

Woo is leading external development, so all our sdks and helping out with documentation and supporting the api channel

drowsy thistle
#

ah icic

kindred maple
drowsy thistle
#

so glad that the Top.gg staff is starting to look after the Open Source side of Top.gg again

deft merlin
#

No we didn't remove your access faith uwu

deft merlin
drowsy thistle
#

xiuh gimme access iara_hmph_angry_pout

kindred maple
#

One message removed from a suspended account.

woven granite
drowsy thistle
#

i have been waiting for over three months

kindred maple
#

One message removed from a suspended account.

tiny shard
#

it’s none of their faults really

kindred maple
tiny shard
#

true

deft merlin
#

It was kind of a thing where we unforcunately had other stuff to prioritize and now we're trying to get more CMs, Mods and BRs involved so it isnt just relying on Team members and CMs to do certain things. Hopefully you'll see a bit of revitalization across a few different places heheCutie

deft merlin
tiny shard
#

swag thank you

kindred maple
#

One message removed from a suspended account.

drowsy thistle
#

considering how the queue now is pretty much near to non-existent onesieKEKW

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

cunning locust
deft merlin
#

you can't react in here?

#

lmao

#

hold on

cunning locust
#

nope

kindred maple
#

One message removed from a suspended account.

woven granite
#

👀

cunning locust
#

i think its to stop regular members

kindred maple
#

One message removed from a suspended account.

woven granite
#

I dont even ahve react

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
#

fixed

cunning locust
#

yay ty

kindred maple
#

One message removed from a suspended account.

drowsy thistle
#

i am a regular

#

time to spam reactions /s

kindred maple
#

One message removed from a suspended account.

tiny shard
#

she meant something

drowsy thistle
#

i see

drowsy thistle
#

who would maintain/continue the development of Top.gg's C++ SDK? since the person maintaining it pretty much left

drowsy thistle
#

poor Brain

deft merlin
#

If we have people interested in helping out with any sdk they can reach out to me or woo nod

cunning locust
#

if needed i am happy helping with node-sdk

#

iirc theres a new library in progress?

#

it's kinda like the one me and burbur started except made a week before KEKW

woven granite
cunning locust
#

i think its that

#

it looked really good

woven granite
#

👀 its crazy that is almost 11 months ago

cunning locust
#

oki

#

well i'll play around with it 🙏

drowsy thistle
drowsy thistle
cunning locust
#

that would be a lot of breaking changes

#

plus not eeveryone wants a lib that basically does it all for you, since this does that including db, etc

#

btw @woven granite - typo there

image.png
#

it's voting-sdk not sfk

woven granite
# drowsy thistle why not merge this with the Node SDK?

The goal with the library was to make voting rewards and reminders to be as simple as possible for developers who dont have experience with databases and webhooks.
I think the hope was that making the library separate it would be as easy as possible for those devs.

#

Realistically we want to do the same thing with a python library.

drowsy thistle
#

bruh

#

its so sad that we have to make separate libraries because lots of bot developers have no experience with webhooks

woven granite
#

Should be something devs learn early on, but many dont really care to learn about them.

drowsy thistle
#

so true honestly

woven granite
#

I hope to make them a bit more access able by making some guides(with some volunteers, and hopefully you all) over the next few months.

drowsy thistle
#

guides?

woven granite
drowsy thistle
#

so

woven granite
drowsy thistle
#

the fact that what is a database is a part of the FAQ is so sad

woven granite
#

👀

#

A lot of bot devs have less than a month of development experience.
When I was learning I didnt understand databases in my first month.

drowsy thistle
#

i knew what they are, i just couldn't understand SQL databases so i used MongoDB purr_lul_haha

#

@woven granite perhaps Top.gg should make an optional Discord webhook option for votes

#

since the first thing that pops up in their mind when they think of webhooks is probably the discord ones

woven granite
#

Realistically, I don't think a dev ever will.

drowsy thistle
#

why is that?

woven granite
#

Well, the message would have to be static.
x voted for y
would be as complex as a top.gg dev would really want to implement. custom messages can become a headache really fast.

I think we're at a scale even being webhooks ratelimits are a concern. which means multiple IPs.

Plus theres just so many other things that would be a priority.

drowsy thistle
#

fairs

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

cunning locust
#

when i started bot development, i had no idea how to code stuff like webhooks

#

so having a lib help handle that easily is good

#

and i literally used a json file as a database, not the best

woven granite
tiny shard
tiny shard
#

probably gonna close this 504 issue if no one else has anything to say about it

woven granite
#

I don't think there's much to say in that issue.
What you said in the PR covers everything.

drowsy thistle
#

owo whats this

kindred maple
#

One message removed from a suspended account.

wary stratus
#

funny role

drowsy thistle
sudden knot
#

👀

drowsy thistle
#

sooo another week has passed and i haven't received any DMs

#

@kindred maple did you get any DMs?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

kindred maple
deft merlin
#

Woo had a family emergency and has been afk for a bit and will be for a bit as well.

kindred maple
#

One message removed from a suspended account.

deft merlin
#

We haven't forgotten about you, i promise.

#

No need to be dramatic KEKW

drowsy thistle
#

does it really have to depend on him?

deft merlin
#

He's the one leading the taskforce so I would like everything to go through him, yes. If something is absolutely 100 percent urgent you can dm me about it.

drowsy thistle
#

but at least he wasnt afk last week, no?

cunning locust
#

i mean tbf theres not really that much of a difference between it being sorted this week vs like next week or the week after smide

wary stratus
#

Especially when a sudden family emergency changes plans entirely

#

Just respect that everyone pepepray

cunning locust
#

^

#

everything here takes time anyways so

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

drowsy thistle
#

fair fair

#

family emergencies are unexpected so i wouldn't blame it on the company

#

so i see why it's delayed, not sure about other things tho

#

they probably lack an engineer actively working on the site

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
#

Right now the external libraries and development/documentation is not a priority for our paid engineering team - the current plan focuses on rewriting Auctions completely from the ground up and then the main website. So the external development is being given to the volunteer team (AKA moderators and the External Development Taskforce) to do what they can when they can - as volunteers. I would love to have the time and people to dedicate to external development properly but we are just too small of a team and have too much of a workload to prioritize the libraries and documentations. So I'm doing what I can, and thats having the volunteers help as much as they can.

#

I am doing what I can to give what I can to our library developers and the libraries and documentation themselves.

#

Thank you for your patience and bearing with us as we do what we can with what limited power we have onionpray

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

drowsy thistle
#

poor that one engineer

#

having to rewrite auctions from scratch

deft merlin
kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

drowsy thistle
#

isn't it 1

#

afaik

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

woven granite
#

To be honest, things happened at the worst possible time.
I agree with you all, things shouldn't just depend on just one volunteer. But before I could get anything ready things stopped.

drowsy thistle
#

i completely understand that

woven granite
#

Over the next few days I'm hoping to get a better picture of my availability so me and xiuh can make a plan.

drowsy thistle
#

thank you for your awareness

deft merlin
#

Hey everyone, just an update. While Woo is away, we're going to have @faint fern step up and take temporarily leadership of the external development things. He should reach out to each Library Developer this week to ask about repos and permissions ThumbsUpSmile we're working first to get a good grasp of who we have active on what repos and then we can move into other tasks.

drowsy thistle
faint fern
#

@royal breach @karmic valley
Heya! Reaching out to you two here as I was not able to Dm you. As you can read above I'm the temporary lead for the external development task force while Woo is away. Just wanted to check what repo(s), if any, you are interested in maintaining/contributing to topggHappy

kindred maple
faint fern
deft merlin
#

Hey everyone o/

You should have received invites or updates to permissions for the repos that you wanted to contribute to. If something seems funky or you need a new invite for any reason, let me know pog_thumbsup otherwise we should be set in terms of permissions

dusky shale
#

Maybe to @tiny shard as they maintain the Top.gg Node.js SDK?

deft merlin
#

That would make sense to me, yes. I can give that if it needed nod

dusky shale
#

I remember when we had to go back and forth for like a week to update a peer dependency in that repository KEKW

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
#

I updated your permissions to maintainer on the java sdk iirc

dusky shale
#

Check the repositories you need permissions on to see if you can edit the descriptions of the repository, if so you're a collaborator

deft merlin
#

you already had it on the other one

kindred maple
deft merlin
#

you already had it on dotnet

#

you had write on java

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
#

everyone should have maintainer on their repos

#

now*

dusky shale
#

I kinda want to be a collaborator on the Top.gg Node.js SDK but only if Berry approves, since they maintain it

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

I don't know I'm totally not a maintainer of JavaScript 👽mmLol

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Well keeping the CI and dependencies and whatnot up-to-date is something at least

kindred maple
#

One message removed from a suspended account.

deft merlin
#

you never know with topgg tbh

kindred maple
dusky shale
#

What's the required .NET or Java version for yours? And possibly breaking dependency updates

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Oh damn, .NET 7 is the latest stable

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Potential breakage with the newer .NET or Java versions, since people tend to update to the newer ones (and often times the newer ones are provided by default)

#

If it works fine with the newer versions then not changing it fine

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Then that's good

#

Though what about dependencies?

kindred maple
dusky shale
#

Oh I meant like any breaking dependency updates available?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

.NET be packing

kindred maple
#

One message removed from a suspended account.

dusky shale
#

The Node.js SDK also wouldn't need any dependencies if Node.js v16 support is removed, though it hasn't reached EoL (End-of-Life) stage so not happening

kindred maple
#

One message removed from a suspended account.

dusky shale
#

Yeah

#

What's funny is that the Ubuntu/Debian repositories provide Node.js v10, so users have to use something like nvm

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Nice, how many dependencies does the Java one have?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

tiny shard
dusky shale
tiny shard
#

those were traumatizing hours

dusky shale
#

💀

deft merlin
#

Added berry to the eslint-config repo

tiny shard
tiny shard
deft merlin
#

IDM, i think volt is pretty trustworthy tbh heh

tiny shard
#

then sure

dusky shale
#

Yeah I'd like to be added, as well as to eslint-config (as the SDK relies on it)

deft merlin
#

Done ThumbsUpSmile

dusky shale
#

Thank you!

dusky shale
#

@deft merlin there doesn't seem to be an npm token set as a secret in either of the top-gg/node-sdk and top-gg/eslint-config repositories so they can't publish to the npm registry via CI/CD

deft merlin
#

I'll have to talk to an engineer about getting that fixed then, since I dont know where we would have that stored.

dusky shale
#

Alright, once you or the engineer get the npm token, create a repository secret like this named NPM_TOKEN in both of those repositories

drowsy thistle
#

yoooooooooo poggythumbsup

image.png
#

thanks @deft merlin

dusky shale
drowsy thistle
#

thanks volt!!!

tiny shard
#

is it broken

#

i can publish anything manually if need be

dusky shale
dusky shale
deft merlin
dusky shale
#

Alright

deft merlin
#

@dusky shale The secret should be accessible to those two repositories and the secret is NPM_AUTOMATION_TOKEN pog_thumbsup

#

If it doesn't work or anything please let me know

dusky shale
dusky shale
#

@tiny shard are you going to create a release or should I do it?

tiny shard
#

i’ll do it

dusky shale
#

Alright

tiny shard
#

action failed

#

forgot to bump

dusky shale
tiny shard
#

okay can you make a PR bumping the version

dusky shale
#

Sure, I'll just run npm version patch

tiny shard
#

sure

dusky shale
tiny shard
#

meta: bump ?

dusky shale
#

Alright

#

Action ran successfully, nice

tiny shard
#

💪

dusky shale
#

@deft merlin I forgot to ask, can you (or an engineer) add me as a collaborator to @top-gg/sdk and @top-gg/eslint-config npm packages? My npm username is voltrexmaster (can't change username to voltrexkeyva due to npm policies ohnodisappear)

deft merlin
#

I can ask yep !

#

Probably won't happen til next week but I'll send a message lol

dusky shale
#

Alright thank you

dusky shale
deft merlin
#

Yeah I'm not sure, I can mention it

kindred maple
#

One message removed from a suspended account.

tiny shard
#

@deft merlin

kindred maple
#

One message removed from a suspended account.

deft merlin
#

I think it's still a thing, I'll have to check

#

Iirc Mac owns the team, I'll check w/ him on Monday

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

drowsy thistle
deft merlin
#

Yes, hopefully. I need access to something and we're trying to figure out how to get it heh

drowsy thistle
deft merlin
#

I'm not sure, i haven't been told specifics

deft merlin
#

I'm traveling this week and will be on vacation the week after so if you reach out, you'll need to expect delayed responses b_nod just a heads up!

kindred maple
#

One message removed from a suspended account.

steady forum
kindred maple
#

One message removed from a suspended account.

steady forum
#

Xiuh prefers doing Auctions support in DMs

wary stratus
#

Yes I can confirm

#

Please DM her for auctions support

#

She likes it

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Rather than one where your dependencies doesn't work with

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

.NET 7 is a thing, but since .NET 8 is coming out in almost 2 months, sure

#

They're not that different either

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Updating to .NET 7 can allow you to change most things that needs updating, so that you most likely won't need to change anything when updating the target version to .NET 8

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

I didn't say anything about that, what I meant is that you can also spend time to do it now to change most things necessary, so you can very easily update to .NET 8 when it comes out

#

But absolutely, you can wait if you want

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Have you tried changing the build target to 7 to see if it breaks anything?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Then sure, waiting would be alright

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Though are you not planning to rewrite some stuff using .NET's new features?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

C/C++

#

😈

kindred maple
dusky shale
#

What new features are you gonna use?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Alright

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Is there a linter you use?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
#

Interesting, there's also SonarLint but I'm not sure if it's better or not

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

dusky shale
kindred maple
drowsy thistle
#

👎

dusky shale
deft merlin
drowsy thistle
dusky shale
deft merlin
drowsy thistle
#

@dusky shale @dusky shale so any ideas on what to improve on it?

dusky shale
drowsy thistle
#

ah aight

#

ill work on the lib internally

dusky shale
#

Funny how 90% of the lib is literally broken

dusky shale
drowsy thistle
drowsy thistle
#

@karmic valley do you happen to still work on the C++ SDK?

dusky shale
drowsy thistle
#

since the code seems a bit messy

karmic valley
#

oooo i can talk here

dusky shale
#

you can rewrite it, it’s messy lol

karmic valley
#

i never had time to start on the C++ SDK

drowsy thistle
#

is the library blocking?

#

i haven't used c++ in a really long time

dusky shale
#

me neither, got rusty on it lol. Forgot the other syntaxes

karmic valley
#

it looks like the C++ SDK uses blocking curl requests

#

i wouldnt have used curl personally

drowsy thistle
#

hmmm

#

what would you use?

#

is D++ blocking too? what does it use for sending requests

karmic valley
#

well i had a more "embrace, extend and extinguish" plan to make it use DPP kekeke i thought better of it though and didnt

karmic valley
drowsy thistle
#

how does that work

karmic valley
#

its written from the ground up to use openssl non blocking i/o and low level berkerly socket calls

#

im not sure how much sense it would be to use it for top.gg api as a standalone thing, as its event loop is tightly tied into D++

#

especially for stuff like coroutines

drowsy thistle
karmic valley
#

but it means you can just do like:

bot.request("lol url", dpp::m_get, [](const auto& result) {
    std::cout << result.http_info.body << "\n";
});

without worrying about stuff blocking

#

theres also bot.co_request

#

which has no callback and uses C++20 coroutines

#

(doesnt block, its like js async/await, you co_await the promise it gives)

drowsy thistle
#

yo that's actually awesome

karmic valley
#

the reason i was going to make it dependent upon D++ is because D++ has basically become the "standard" library for discord on C++, stuff like sleepy discord is dead af

drowsy thistle
#

never knew c++ had coroutines

karmic valley
#

its new

#

and under the hood its awful

#

they give you this raw horrid api and go "write something nice on top" and then you have to implement a user side thing

drowsy thistle
karmic valley
#

yeah, but thats C, not C++, different language

drowsy thistle
#

i wouldn't make my Rust SDK only biased towards Serenity

karmic valley
#

might as well make a different api client for it

#

a C compatible api client would be like, void* ptrs, raw pointer ops, malloc, free, and C style function callbacks

#

also has very small userbase

#

in the rust land, there are many very popular libs

#

the only thing i'd say definitely would be worth avoiding in C++ api is... dont pull in boost or asio. theyre absolutely massive

#

certainly not boost::beast

drowsy thistle
#

frfr

#

boost is one of the thiccest libraries out there

karmic valley
#

i hate boost

drowsy thistle
#

same

#

installing and linking it is hell

karmic valley
#

and i hate how shitty bits of it keeps finding their way into std::

#

like chrono

drowsy thistle
#

yet it's everywhere on stackoverflow threads

karmic valley
#

i couldnt speak here before

#

did i gain a role or were permissions fixed

drowsy thistle
#

the latter

karmic valley
#

ah

dusky shale
karmic valley
#

yeah

#

i cant vouch for its performance

#

not measured it

drowsy thistle
karmic valley
#

i bet its based on asio, where all the C++ trydiehards sing its praises as the ultimate in scalability.... might be for specific use cases, but ive always managed to outclass it with raw and basic berkerly socket api

drowsy thistle
#

wth is beast for anyway

karmic valley
#

take away the 30 layers of java-like abstraction, and oddly enough things are fast

dusky shale
#

cppreference lists quite a lot of libraries for HTTP(/2|S) clients and stuff

karmic valley
#

beast is a http library in boost

drowsy thistle
#

anyway, what http library should the C++ SDK use

karmic valley
#

tbh, you dont need http2, http3, and all the other junk. 100% of webservers speak http1.1 with chunking just fine and you get no advantage using anything later for REST

drowsy thistle
#

there are a lot of C/C++ http libraries out there

karmic valley
#

thats why i made my own, didnt want all the useless cruft

#

the one i used before that, which i strongly recommend is cpphttplib

#

header only, pretty light, and very fast

#

BUT it only has a blocking api, you have to wrap it in a threadpool or something

drowsy thistle
#

sadge

karmic valley
#

in D++ the requests are marshalled through a threadpool so it doesnt matter if the underlying lib blocks

drowsy thistle
#

so write one from scratch mimu_thisisfine

dusky shale
deft merlin
karmic valley
#

could probably lift it and reuse it tbh

drowsy thistle
#

i mean the Rust SDK also has its own http module that i wrote from (near) scratch onesieKEKW

karmic valley
#

that's what powers co_request

#

tbh making a http request isnt hard

#

its the setup of the https session thats the pain

drowsy thistle
#

true

karmic valley
#

its about 50 lines

#

if you do proper error checking

dusky shale
#

cpp-netlib and cpp-httplib are also pretty good

kindred maple
#

One message removed from a suspended account.

steady forum
#

You should follow up with Xiuh about that. She's managing all the volunteer task stuff.

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
#

Whats the ID of the lib dev bot?

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

kindred maple
#

One message removed from a suspended account.

deft merlin
#

I need to get added to the team then i can invite you

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

deft merlin
#

yeah youre set to a viewer

kindred maple
#

One message removed from a suspended account.

deft merlin
#

i meant update your permissions, not invite you sorr

#

i will nod

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

#

One message removed from a suspended account.

wary stratus
#

Why can't you even get the token

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

wary stratus
#

What does it tell you when you try to

kindred maple
#

One message removed from a suspended account.

#

One message removed from a suspended account.

wary stratus
#

Try accessing/webhooks

deft merlin
#

I think its cause theyre set to viewer and not moderator

kindred maple
#

One message removed from a suspended account.

wary stratus
#

Are you also in the team?

kindred maple
#

One message removed from a suspended account.

deft merlin
#

try now