Skip to content

Conversation

JFreegman
Copy link
Member

@JFreegman JFreegman commented Jan 16, 2024

This fixes an issue with events where we try to make queries on peers that no longer exist internally


This change is Reviewable

@JFreegman JFreegman added the cleanup Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature. label Jan 16, 2024
@JFreegman JFreegman added this to the v0.2.19 milestone Jan 16, 2024
@JFreegman JFreegman force-pushed the group_delete_iteration branch from d5352bd to 2e73701 Compare January 16, 2024 17:34
Copy link

codecov bot commented Jan 16, 2024

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (16809dc) 66.92% compared to head (21a8ff5) 66.96%.

Files Patch % Lines
toxcore/group_chats.c 84.61% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2573      +/-   ##
==========================================
+ Coverage   66.92%   66.96%   +0.04%     
==========================================
  Files         148      148              
  Lines       30271    30275       +4     
==========================================
+ Hits        20259    20275      +16     
+ Misses      10012    10000      -12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@iphydf iphydf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kind of meh, but alright. I don't see a better solution.

…tion

This fixes an issue with events where we try to make queries on peers
that no longer exist internally
@JFreegman JFreegman force-pushed the group_delete_iteration branch from 2e73701 to 21a8ff5 Compare January 16, 2024 18:12
@toktok-releaser toktok-releaser merged commit 21a8ff5 into TokTok:master Jan 16, 2024
@JFreegman JFreegman deleted the group_delete_iteration branch January 16, 2024 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature.
Development

Successfully merging this pull request may close these issues.

3 participants