Cassandra McCarthy
Cassandra McCarthy
I think it would be good to just link to the docs in a follow up, yes
There could still be a future-proofing code change made here. We could make `entity_metadata: Option`, and add a check on each method that modifies it, which nulls the field itself...
Marking as draft until the gateway refactor merges
I tend to agree with this. I have one design question though: how should we report multiple errors from different parts of the builder? Say the content is too long...
> Builders in frameworks like `http` just don't report multiple errors, they just [report the first error encountered](https://docs.rs/http/0.2.5/src/http/request.rs.html#779-789). It seems we also have precedence for this in the `EmbedBuilder`. This...
I can get to this by the weekend.
Could you update this for https://github.com/discord/discord-api-docs/pull/5093? Two birds one stone, I figure it's in scope
Since the entire crate is refactored by #1809, I'll add it to that issue and close the component ones.
Any progress on this?
I like these! I must ask, why does the function wrap to two less than the reported character length?