I agreed with @jbpasquier to only work on estimated and approved issues. I'm not sure what to estimate for an issue like this without investigating it. What is the expected process for this?
@MattJ@XaviFP Yes, sorry I thought it was clear but bug-s1 means we'd finance it because it is critical.
@jbpasquier Well... people can't talk with each other anymore when they lose their history, the messages they were supposed to read are not displayed, therefore the app literally ceases to function for them.
@alexbourlier From the issue, it stands for an history loss on one one-to-one conversation, I don't experience anything similar on my side. I have no clue if it impact other people or even if your contact also face it. Standing on that, I don't feel this issue as an s1, but you and @rachel are masters on this. :-)
This might lead to errors while fetching those in the backend and consequently generating those 404 to the GET requests to https://api.community.startinblox.com/circles/XXX/
In case it helps I'm getting {"detail":"Not found.","@context":"https://cdn.happy-dev.fr/owl/hdcontext.jsonld"} on GET request to https://api.community.startinblox.com/circles/14/
I think I would hesitate to put it as s1, but as @alexbourlier is the master of budget I trust he is using s1 carefuly :) That being said, unless checking all new issues, there is no way any member of the team can currently know when an s1 is created. What about adding to the process that people get pinged? At least maybe JB + myself? So we can estimate if we need to communicate it more widlely?
Ok, had a quick look into this just now at the request of @jbpasquier who is also experiencing the issue.
The good news is that history is there - Alex's archive spans from April to today, so it's not a question of it being lost or missing.
@jbpasquier reported the issue to me at 18:35 UTC, but I see no history queries from jbpasquier@community.startinblox.com since 14:51 UTC. That query returned 30 results from Prosody to Converse.js.
It again seems to me that there is some problem with Converse.js not reliably fetching history. We know one bug is when there are no displayable messages in the results (we have reduced many cases of this, but I can't guarantee it will never happen - undisplayable messages are a natural part of XMPP to ensure e.g. reliable delivery reports).
@XaviFP is this something you can reproduce, can you see if Converse.js is correctly querying when it should, displaying the result and requesting further results when necessary?
I have the feeling we are unable to solve any bug in less than 1 or 2 months on the chat component. The same is happening with #161 (closed), and we are not releasing any feature requests in the meantime. That's not acceptable.
I understand we needed to update ConverseJS, I understand there are performance issues, I understand personal emergencies and there might be other things that I'm not aware of, but we can't afford to solve critical bugs in months, this needs to improve and by a significant factor. We are losing our best advocates of our federated app project because they doubt we'll be able to have this thing working one day. They kep experiencing the same bugs and they stop believing me we'll fix them one day. We won't go anywhere if we don't keep them on board, and I start to worry a lot that we will.
Can we have a call to discuss this?
I'm available anytime, any day.
We need to be faster at solving bugs on the chat component otherwise we just won't make it through 2021.
@jbpasquier I dared to switch this one back as a bug-s1, cf. the labels description. This is a critical broken feature with no workaround. I can't talk to @Cyrilthiriet anymore on Hubl and have to default to Whatsapp for doing so.
It could be related to the cause of #193 (closed). There are some similar behaviors like the need of re-scrolling, after the last fix to avoid Converse from scrolling down after a history fetch.
Need to investigate further, but all these issues seem related to me. Or at least I believe are part of the same chain reaction.