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

payment.amazonGiftCard.info = Inconsistent with wiki about Message Field #5726

Closed
Liisachan opened this issue Sep 25, 2021 · 6 comments · Fixed by #5727
Closed

payment.amazonGiftCard.info = Inconsistent with wiki about Message Field #5726

Liisachan opened this issue Sep 25, 2021 · 6 comments · Fixed by #5727

Comments

@Liisachan
Copy link

Description

In \core\src\main\resources\i18n\displayStrings.properties (and its translations), payment.amazonGiftCard.info states: “you must include the trade ID in the gift card''s message field. Please see the wiki”.

In https://bisq.wiki/Amazon_eGift_card#Step_2._Customize_eGift_card however, it is said that the Message field should be left blank.

This can confuse the user as to whether or not they should include the trade ID there. Maybe writing the ID there is usually not necessary nor desirable...?

It's not really a bug - more like a documentation issue.

Version

v1.7.3

@pazza83
Copy link

pazza83 commented Sep 25, 2021

I removed the reference from the image.

I cannot find any other reference to trade ID on the page.

Let me know if I have missed anything.

@Liisachan
Copy link
Author

Sorry if I was not clear. I'm talking about source code:

Bisq will show the BTC seller''s email address or phone number where the gift card should be sent, and you must include the trade ID in the gift \

@pazza83
Copy link

pazza83 commented Sep 25, 2021

Hi @m52go are you able to remove this from Bisq source code?

@Liisachan
Copy link
Author

Maybe not just removing, but explicitly advising the user not to say anything about btc there? While sending a gift card to your friend is perfectly legal, commonly done by millions of amazon users, it may be safer to keep a low profile to avoid unwanted attentions...?

@ripcurlx
Copy link
Contributor

Maybe not just removing, but explicitly advising the user not to say anything about btc there? While sending a gift card to your friend is perfectly legal, commonly done by millions of amazon users, it may be safer to keep a low profile to avoid unwanted attentions...?

Good catch! I think just removing the trade id reference should be fine as during the trade this is mentioned explicitly anyways.

portfolio.pending.step2_buyer.refTextWarn=Important: when making the payment, leave the \"reason for payment\" field \
empty. DO NOT put the trade ID or any other text like 'bitcoin', 'BTC', or 'Bisq'. \
You are free to discuss via trader chat if an alternate \"reason for payment\" would be suitable to you both.

@ripcurlx
Copy link
Contributor

I just checked the remaining payment methods and only found it for halcash which is fine as there it is sent via text message.

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 a pull request may close this issue.

3 participants