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

Replace the Get Support button with Open Trader Chat until trade period is over #4099

Merged
merged 1 commit into from Apr 10, 2020
Merged

Conversation

ghost
Copy link

@ghost ghost commented Mar 30, 2020

Showing an Open Trader Chat button until the trade period is over will reduce the number of unnecessary support tickets and encourage trader-to-trader dialogue. If the trade timer expires without completing the button changes (as before) to open a mediation ticket.

Implementation of this feature requires the button in TradeStepView to notify its parent TradeSubView which in turn notify its parent PendingTradesView that trader chat is to be opened. Basically a callback interface is passed two levels down the GUI hierarchy.

Fixes #3801

@ghost
Copy link
Author

ghost commented Mar 30, 2020

example:

image

@ripcurlx ripcurlx added this to the v1.3.0 milestone Apr 1, 2020
@ripcurlx
Copy link
Contributor

ripcurlx commented Apr 2, 2020

@jmacxx I'm having a look at it right now. Please don't make any changes to translations except the English source file, as they are overwritten shortly before every release.

@ghost
Copy link
Author

ghost commented Apr 2, 2020

@ripcurlx Ok, I have restored the language files to their original (de, es, pt, pt-br).

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NACK - Please see my remarks

ACK functionality wise

Showing an `Open Trader Chat` button until the trade period is over will
reduce the number of unnecessary support tickets and encourage
trader-to-trader dialogue.  If the trade timer expires without
completing the button changes (as before) to open a mediation ticket.

Implementation of this feature requires the button in TradeStepView to
notify its parent TradeSubView which in turn notify its parent
PendingTradesView that trader chat is to be opened.  Basically a
callback interface is passed two levels down the GUI hierarchy.

Fixes #3801
@ghost ghost changed the title Replace the Get Support with Open Trader Chat until trade period is over Replace the Get Support button with Open Trader Chat until trade period is over Apr 7, 2020
@ghost ghost mentioned this pull request Apr 7, 2020
@ghost ghost requested a review from ripcurlx April 9, 2020 21:42
Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

Tested it again on Regtest and everything works as expected.

Before trade period is over it will show

Bildschirmfoto 2020-04-10 um 16 15 30

and after the trade period it will show

Bildschirmfoto 2020-04-10 um 19 51 15

@ripcurlx ripcurlx merged commit edc4df1 into bisq-network:master Apr 10, 2020
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.

Emphasize that traders engage in trader chat to solve trade problems
1 participant