asgiom / cr8-brm-mobile-issues

0 stars 1 forks source link

[BUG]: Displayed QR code different to shared QR code... #134

Open t-jacobi opened 4 years ago

t-jacobi commented 4 years ago

Describe the bug The QR code that will be displayed on the screen (in app) is different to the QR code created due to the share button.

To Reproduce Steps to reproduce the behavior (Variant 1):

  1. Go to 'Receive'
  2. Enter a '6' into the amount and remove it again
  3. Scan the QR code in app to see the content - save the content
  4. Click on Share account and also scan the QR code and save the content
  5. in my case by using the share account functionality added a &amount=6 string to the URL behind.

Steps to reproduce the behavior (Variant 2):

  1. Go to 'Receive'
  2. Scan the QR code in app to see the content - save the content
  3. Click on Share account and also scan the QR code and save the content
  4. in my case by using the share account functionality only the link https://casinocoin.org is stored as content in the QR.

Expected behavior The same QR code displayed inapp should be used instead of a new generated one...

Screenshots

IMG_4398 Content: https://casinocoin.org/send?to=cacmgvVWFbfTvR8RUqzEyYVAQsP74EYb6C&token=CR8

IMG_4399 Content: https://casinocoin.org/send?to=cacmgvVWFbfTvR8RUqzEyYVAQsP74EYb6C&token=CR8&amount=6

Smartphone (please complete the following information):

Additional context

t3nshi commented 4 years ago

Actually it seems that It is taking the accountID well... after testing a big chunk of the day today I realized that the QR code shared is slightly outdated compared to the one we are currently changing on the receive page. Do this so you can see: when I put the 1234 destination tag on the receive page and I scan the QR from different device, I will get everything right except the destination tag is 123, and I added now a 5 so it is 12345 and scanned is 1234 so I removed the 5 and now it is 12345, that led me to the right track... but so far I cannot solve it just confirm it.