ehubscher / briarproject_390

Concordia University Mini-Capstone (i.e. SOEN 390) project for Team La Famiglia. The Open-Source Project forked is an Android-based secure messaging app called Briar.
Other
4 stars 1 forks source link

BUG: Fix only one message can be sent #147

Open Winterhart opened 6 years ago

Winterhart commented 6 years ago

Pined message works well but we need to fix a little bug with messages.

Problem:

It seems like the Activity is not refreshing ? I know this bug it present since pull #140 We can solve the bug in pull request #144 maybe ?

A video of the behavior: https://youtu.be/ARlPkH4cBh8

In this video, the other user only got two messages: 'Coucou' and 'Coucou2'

Git Bisect Document to find where the bug was introduced: https://docs.google.com/document/d/14IQcQD7U3-PspVl1s4JL1HNWPkp5NbvES6V_MlqPf8g/edit?usp=sharing

Winterhart commented 6 years ago

The bug was introduced: https://github.com/jhubscher/briarproject_390/commit/b72c44cc00353fc533f44d79f96978758c7b72c5 or in https://github.com/jhubscher/briarproject_390/commit/ba24d799c63d995eac517756a389a165d9e65b16 but this commit doesn't compile

philbeaudry commented 6 years ago

This issue will be very hard to fix given our development tool kit. Only one developer with multiple devices for continuous testing.

Winterhart commented 6 years ago

Actualy you can test it with one device by creating fake data and sending messages to those fake contact...