Open claucece opened 5 years ago
Hey, I found a inconsistency which is worth looking since it's related to user feedback. The change seems to be introduced by c276bfa7
There are two conflicting messages showing up in the conversation window displaying OTR conversation status updates. In the first case, one saying "Not Private" and the other says "Private". In the second case, it shows "Not Private" and the usual "Private conversation started".
This seems to be a intermediate state that never shows up independently as far as I tested. Also there's the "Finished" state that I never managed to display in a conversation.
I'm not sure if we should fix this logic or remove this at all, since it seems old and buggy and we already have a label displaying the OTR conversation status. Thoughts @claucece @DrWhax @olabini ?
Here's the code https://github.com/otrv4/pidgin-otrng/blob/master/gtk-dialog.c#L2765
First case - steps to reproduce:
Second case - steps to reproduce:
Hey!
Thanks for looking at this.
I have never seen this message: 'The privacy status of the current conversation is now: Private' nor 'The privacy status of the current conversation is now: Not Private' in any case. The status is only show in the bar for me.
The change seems to be introduced by c276bfa
I don't see where this was introduced.. can you point the line and file? This is a huge commit that changes a lot of things..
If you are referring to this:
switch (current_level) { case TRUST_NOTPRIVATE: status = ("Not Private"); break; case TRUSTUNVERIFIED: status = ("Unverified"); break; case TRUSTPRIVATE: status = ("Private"); break; case TRUSTFINISHED: status = ("Finished"); break; }
from what I remember that is used to changed the appearance of buttons and other stuff, so I'll say don't remove that.
This seems to be a intermediate state that never shows up independently as far as I tested. Also there's the "Finished" state that I never managed to display in a conversation.
The "Finished" state is reached on the menu bar when the other party ended the conversation:
I pushed something @giovaneliberato .. Now, at least I see the status.. check if it solves for you :).
I don't see where this was introduced.. can you point the line and file? This is a huge commit that changes a lot of things..
I found it hard to track too, but the point I want to make is that this is a very old change that we inherit.
I have never seen this message: 'The privacy status of the current conversation is now: Private' nor 'The privacy status of the current conversation is now: Not Private' in any case. The status is only show in the bar for me.
Hmm, that's weird. For me now it's being displayed three times for the use case 1 I described above
from what I remember that is used to changed the appearance of buttons and other stuff, so I'll say don't remove that.
Actually the logic for the OTR button is implemented here in gtk-dialog.c:1232.
This switch case located at gtk-dialog.c#L2743 is used only for this message that is shown in the message box. Line 2766
contains the template being used in these messages.
So my suggestion to remove this is because: 1) We already have two places informing the current status:
Unverified/Private conversation started. Your client[..]
and Private conversation lost.
that shows up in the conversation window.
2) It seems like it working inconsistently between us two, maybe someone else can try it.
3) It contains links that #95 aims to removeWhat do you think @claucece?
Hey!
Actually the logic for the OTR button is implemented here in gtk-dialog.c:1232. This switch case located at gtk-dialog.c#L2743 is used only for this message that is shown in the message box. Line 2766 contains the template being used in th
I'm not referring to the OTR button, but to the buttons of the OTR menu. This kind of trust level is used in several places to define which button gets active, etc. But I see for this case is only for the information.
The messages Unverified/Private conversation started. Your client[..] and Private conversation lost. that shows up in the conversation window.
I actually think that in this case it seems to be ok. In the course of a long conversation, I don't think an user is going to only look at the bottom toolbar to check the status. Maybe we should unified them into a more explicit 'Unverified/Private conversation started.'.. I never liked those texts anyhow.
It seems like it working inconsistently between us two, maybe someone else can try it.
Is it still working inconsistency after commit d452471d889652a1dc66846bf2bf835106dcb64b ? It is showing and working ok now for me.. not for you?
It contains links that #95 aims to remove
Those can be easily removed.
Thanks!
So, after thinking for a while.. I think it makes sense to remove them. But I think we should create better texts that unify the ideas of 'Private conversation started. Your client is not logging this conversation and The privacy status of the current conversation is now: Private'. Right now, both texts are not clear on what they mean. I'll push a commit removing the hyperlink and then, if you can, @giovaneliberato you can remove them. Thanks!
So, after thinking for a while.. I think it makes sense to remove them. But I think we should create better texts that unify the ideas of 'Private conversation started. Your client is not logging this conversation and The privacy status of the current conversation is now: Private'. Right now, both texts are not clear on what they mean.
Agree! I'll remove these messages and we work on improving the texts.
Is it still working inconsistency after commit d452471 ? It is showing and working ok now for me.. not for you?
No :( For me it is showing multiple times as in the screen shot. I'm wondering why we'd see such differences.
For me it is showing multiple times as in the screen shot. I'm wondering why we'd see such differences.
Is because they are using pointers for comparison, which makes it system dependent.
Hey, found some inconsistencies with the flow of this test case, the suggested flow was updated in the Test Case Wiki, the modifications are related with:
Awesome @MauroVelasco ! Thanks so much!
The complete list of files are created when the plugin pidgin-otrng
is disabled and enabled again.
But if the plugin is enabled and the files are previously deleted, some files are not created:
This is the list of files created:
This is the list of missing files:
Pidgin is closed unexpectedly in this step:
* Ensure that "Start private conversation" is enabled, while "End private conversation and "Authenticate buddy" are not enabled"
* Click on "Start private conversation"`
Ensure that the test case in https://github.com/otrv4/pidgin-otrng/wiki/Test-Case---Interactive-path works correctly. In the process of doing that, fill in the documentation in the wiki with more details where necessary.