Inconsistent Design - Emote/GuildEmote Deletion & Modification

[Still34]

At the moment, we have 2 separate methods under IGuild called ModifyEmoteAsync and DeleteEmoteAsync, which doesn't seem to fit the modification/deletion verb design we have going on in the lib, namely ModifyAsync and DeleteAsync.

Shouldn't we make this Emote.ModifyAsync and Emote.DeleteAsync or under GuildEmote instead?

[FiniteReality]

All of our Emote classes are in Core, since they're basically container types for data. Adding ModifyAsync/DeleteAsync would require a large breaking change to move them over into at least Rest

[Still34]

Perhaps 3.0 then?