Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Message received receipts stateless #58

Merged
merged 1 commit into from
Aug 22, 2016

Conversation

GrayHatter
Copy link

@GrayHatter GrayHatter commented Aug 20, 2016

This change is Reviewable

@nurupo
Copy link
Member

nurupo commented Aug 21, 2016

:lgtm:


Reviewed 6 of 6 files at r1, 5 of 5 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.


Comments from Reviewable

@GrayHatter GrayHatter force-pushed the stateless_msgrecpts branch 3 times, most recently from 87495ea to 74b47fa Compare August 22, 2016 09:10
@iphydf
Copy link
Member

iphydf commented Aug 22, 2016

:lgtm: what a shitty tool reviewable is :\ and it's still the best.


Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@iphydf
Copy link
Member

iphydf commented Aug 22, 2016

Reviewed 6 of 6 files at r1, 5 of 5 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.


Comments from Reviewable

@iphydf
Copy link
Member

iphydf commented Aug 22, 2016

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.


Comments from Reviewable

@iphydf iphydf merged commit a6f2e95 into TokTok:master Aug 22, 2016
@GrayHatter GrayHatter deleted the stateless_msgrecpts branch August 24, 2016 06:44
@iphydf iphydf modified the milestone: v0.0.1 Nov 6, 2016
iacore pushed a commit to iacore/c-toxcore that referenced this pull request Jul 14, 2024
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants