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

Revert "Optimized counter increment logic for 1-1 chat messages" #4059

Closed
wants to merge 1 commit into from

Conversation

alwx
Copy link
Contributor

@alwx alwx commented Sep 25, 2023

Reverts #3889

@alwx alwx requested a review from churik September 25, 2023 12:50
@status-im-auto
Copy link
Member

status-im-auto commented Sep 25, 2023

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 465aea5 #1 2023-09-25 12:54:31 ~3 min linux 📦zip
✔️ 465aea5 #1 2023-09-25 12:55:50 ~4 min ios 📦zip
✔️ 465aea5 #1 2023-09-25 12:56:12 ~5 min android 📦aar
✔️ 465aea5 #1 2023-09-25 13:23:42 ~32 min tests 📄log

@alwx alwx requested a review from vitvly September 25, 2023 12:55
vitvly pushed a commit to status-im/status-mobile that referenced this pull request Sep 25, 2023
@vitvly
Copy link
Contributor

vitvly commented Sep 25, 2023

I was able to add contacts and send messages between them using status-mobile pointing to this PR.

@vitvly
Copy link
Contributor

vitvly commented Sep 25, 2023

However, this doesn't seem to be the root cause of issues here status-im/status-mobile#17344 (comment), so most likely this particular change (counter increment logic) is not the culprit.

@alwx alwx closed this Oct 12, 2023
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