NetCordDev / NetCord

The modern and fully customizable C# Discord library.
https://netcord.dev
MIT License
65 stars 10 forks source link

Added documentation to GatewayIntents.cs, GatewayClient.cs, ShardedGatewayClient.cs and GatewayOpcodes.cs #20

Closed Red-K0 closed 5 months ago

Red-K0 commented 6 months ago

Fixed the brace

KubaZ2 commented 6 months ago

What about using the see tag to mention events from GatewayClient instead of using the Discord raw event names?

Red-K0 commented 6 months ago

What about using the see tag to mention events from GatewayClient instead of using the Discord raw event names?

I thought about it, but I couldn't seem to find any centralized way to reference them all without having to tag random members across NetCord. If it does exist, I'll use it though. (I could also add one)

Red-K0 commented 6 months ago

To clarify on my last comment, I was referring to newer events such as polls.

KubaZ2 commented 6 months ago

I would remove the polls intents from this pr and move them to https://github.com/KubaZ2/NetCord/pull/18.

Red-K0 commented 6 months ago

Sure, moving them after I update the see tags then.

Red-K0 commented 6 months ago

<see> tagged literally everything.

Red-K0 commented 6 months ago

Added more documentation in the meantime, all events in GatewayClient should be covered now.

Red-K0 commented 6 months ago

All bullet points converted to XML tags.

Red-K0 commented 6 months ago

Covered pretty much all the comments so far.

Red-K0 commented 6 months ago

All intents and their requirements/influences laid out in the docs, I'll add more I can find, but the bulk of them are ready.

Red-K0 commented 6 months ago

Documented pretty much every required intent and its influence.

Red-K0 commented 6 months ago

Sure, replaced them all.

Red-K0 commented 5 months ago

Instead of being pedantic about it, added the suggestion, but fixed the casing of The to the.

Red-K0 commented 5 months ago

Changed the documentation for it

Red-K0 commented 5 months ago

We should probably try and finalize this soon

Red-K0 commented 5 months ago

Added IPartialMessage changes.

KubaZ2 commented 5 months ago

Could you also add xml docs for ShardedGatewayClient?

Red-K0 commented 5 months ago

Sure