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

Add Capitual payment method #5501

Merged
merged 8 commits into from
Jul 19, 2021
Merged

Conversation

BtcContributor
Copy link
Contributor

Implement bisq-network/growth#246

Waiting for final Info text to be suggested by @pazza83

In the meantime, if you have any feedback on this please let me know :)

capitual

@BtcContributor
Copy link
Contributor Author

@pazza83 do you have any update on final info text to add? Thanks!

@pazza83
Copy link

pazza83 commented Jun 5, 2021

Hi @BtcContributor still waiting for my feedback on this comment from @m52go

I do not think any special information is needed. Let me know if you think I am overlooking anything

@BtcContributor BtcContributor requested a review from ripcurlx June 14, 2021 14:48
@BtcContributor BtcContributor marked this pull request as ready for review June 14, 2021 14:49
@BtcContributor
Copy link
Contributor Author

Applied feedbacks from @KaiWitt (thanks!) and added text suggestion from @pazza83

I think now it is ready to be moved to final PR and no draft anymore :)

@@ -3466,7 +3467,7 @@ payment.amazonGiftCard.info=To pay with Amazon eGift Card, you will need to send
- try to use creative, believable text for the gift card''s message (e.g., "Happy birthday Susan!") along with the trade ID (and use trader chat \
to tell your trading peer the reference text you picked so they can verify your payment)\n\
- Amazon eGift Cards can only be redeemed on the Amazon website they were purchased on (e.g., a gift card purchased on amazon.it can only be redeemed on amazon.it)

payment.capitual.info=Include any special instructions for the payment method that both buyer and seller might need to be aware of
Copy link
Contributor

Choose a reason for hiding this comment

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

FYI this line actually shows in the software...it should probably be removed.

Screenshot from 2021-06-15 23-48-32

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So, no popup or different text?

Copy link
Contributor

Choose a reason for hiding this comment

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

I would say if there's no text worth putting there, there should be no pop-up.

Copy link

Choose a reason for hiding this comment

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

Yes no special text needed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove this unused translation.

@BtcContributor
Copy link
Contributor Author

@ripcurlx @pazza83 now it should be good, popup removed

@ripcurlx
Copy link
Contributor

@BtcContributor I completely missed this PR for the v1.7.2 release. Is this PR ready to be reviewed and tested?

@ripcurlx
Copy link
Contributor

@BtcContributor I completely missed this PR for the v1.7.2 release. Is this PR ready to be reviewed and tested?

Ok - I'll test and cherry-pick it for the release if everything looks fine tomorrow.

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 (with my last two commits)

Bildschirmfoto 2021-07-19 um 13 52 33

Bildschirmfoto 2021-07-19 um 13 52 18

I assume the CAP code is sufficient for trading @pazza83 ?

@ripcurlx ripcurlx merged commit 3512313 into bisq-network:master Jul 19, 2021
@ripcurlx ripcurlx added this to the v1.7.2 milestone Jul 19, 2021
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.

5 participants