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

fix(chatform): Remove assertion that breaks friend information call. #478

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nickolay168
Copy link

@nickolay168 nickolay168 commented Jan 27, 2025

In this PR we are removing the assertion that the logging is on, when user tries to get the friend information. This assertion causes qTox failure when the logging is switched off.


This change is Reviewable

@github-actions github-actions bot added the bug Bug fix for the user, not a fix to a build script label Jan 27, 2025
src/persistence/history.cpp Outdated Show resolved Hide resolved
@nickolay168 nickolay168 requested a review from iphydf January 27, 2025 05:16
@iphydf iphydf added this to the v1.18.3 milestone Jan 29, 2025
@nickolay168 nickolay168 force-pushed the nickolay168_fix_get_details branch from d6c11b3 to f1540fa Compare January 29, 2025 03:55
@iphydf
Copy link
Member

iphydf commented Jan 29, 2025

Thanks, can you rebase on master? Then we can merge it.

@nickolay168 nickolay168 force-pushed the nickolay168_fix_get_details branch from f1540fa to 1a04870 Compare January 30, 2025 06:55
@nickolay168
Copy link
Author

Thanks, can you rebase on master? Then we can merge it.

Done! Thank you!

@iphydf
Copy link
Member

iphydf commented Jan 30, 2025

Can you sign your commit? We can't merge otherwise. I can't sign it for you because your fork is of qTox/qTox, not TokTok/qTox.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug fix for the user, not a fix to a build script
Development

Successfully merging this pull request may close these issues.

2 participants