ircv3 / ircv3.github.io

IRCv3 website
https://ircv3.net/
Other
87 stars 86 forks source link

Clarify what it means for clients to support `batch` #177

Open DanielOaks opened 6 years ago

DanielOaks commented 6 years ago

It's pretty explicit when servers support batches. Q: Do they send batches. A: Y/N.

However, when it comes to clients... it's much less explicit. Some example questions we could apply:

I dunno, it's hard to say when a client really supports 'batches'. What I think makes the most sense is for the actual batch column on the standard support page to be green when the client requests / can request batch (and handle messages in batches without dropping them or similar), and to create a similar batch-types support page similar to the SASL mechs support page, detailing exactly which batches are supported.

The above is what I'll be going by for the future, and I'll make that batch types support page sometime soon.

DarthGandalf commented 6 years ago

To clarify: are you talking about the spec, or about support tables? Spec is vague by design, to allow clients make use of batches as they see fit. Regarding the support tables, I agree, it would be useful to know how exactly each client handles batches.

prawnsalad commented 6 years ago

Echoing @DarthGandalf, clients may want to handle batches in their own way.

However, as long as a client can request + handle batch commands then they they do support batch. If a client has no need to send batch requests then the support pages showing this may come across as penalising in comparison to others for no good reason.

SadieCat commented 6 years ago

IMO supporting batch shouldn't involve supporting any particular batch type. It should just be requesting and supporting the infrastructure for receiving and handling batches.

We can have a separate support table for specific batch types.

DanielOaks commented 6 years ago

Coolcool, in that case we're all pretty much on the same page, can just keep doing this in the future. I'll leave this issue open 'til #45 is resolved and add a note somewhere so we know this going forward.

edit: @DarthGandalf just the support tables yeah