Closed Schmiddiii closed 7 months ago
Thanks a lot! We could definitely introduce a convenience wrapper type for avatars if you think it's worth it.
I have now updated this PR. A few notes:
type AvatarBytes = Vec<u8>
.I am still unsure if there exists some message that notifies when a profile/avatar gets out-of-date.
Note from WF chat: Profile updates seem to be done by WF and SA on a schedule. While presage does not yet have such a schedule, we could certainly do something similar. But I think this can also be done in another MR, I would therefore call this mergeable (if you don't have any comments on it).
See also:
Thanks a lot for your work! I fixed some cosmetic things and some clippy
warnings, and will merge it now.
A few TODOs are left which I am unsure about:
Vec<u8>
, e.g.Bytes
?