search

MXCzkEVM / datadash-wallet

Other
0 stars 3 forks source link

New changes to app #287

Open reasje opened 2 months ago

reasje commented 2 months ago

Changes : Change Moonchain Wallet to MoonBase Increase text logo quality Update MNS pages

SanghamitraBhowmick1993 commented 2 months ago
  1. Please help fix the below. Clicking customer support --> Learn More, i should see moonchain instead of mxc
image

  1. Where exactly will I see moonbase, could you give me an example?

  2. How about mns ip's, should they also have .moonchain instead of .mxc

    image

  3. Also what exactly is the change in MNS. Could you please list it down for my testing

  4. So, should I follow this figma for all the UI/UX validations? https://www.figma.com/design/Z1vdHudqPwFHudfS9B9ZCH/Moonchain-dApps?node-id=494-167&node-type=canvas&t=oQCOjlAqvDxjxBQW-0

@reasje

reasje commented 2 months ago

@SanghamitraBhowmick1993

  1. MoonBase is the app name only. image
  2. No Still MXC (Plz confirm with Sheen)
  3. MNS on Moonchain app only related splash screens are change telegram-cloud-photo-size-4-5990255853862765330-y telegram-cloud-photo-size-4-5990255853862765329-y
reasje commented 2 months ago

@SanghamitraBhowmick1993

  1. The link will be updated
SanghamitraBhowmick1993 commented 2 months ago

Thank you for the confirmation @reasje

  1. For a newly created account, an error screen is visible when clicked on 'View Other txs' @reasje
image image
  1. Some of the links of exchanges below are not working . Maybe we need to update the url?
image

cc: @sheenhx

reasje commented 2 months ago

@SanghamitraBhowmick1993 Your welcome.

  1. For accounts with no transaction record It's expected to see that error message, If you have transaction record and still see that page then that is the issue.
  2. All working for me, There is a possibility the exchange is banned in your region, Plz attach the error you encounter.
SanghamitraBhowmick1993 commented 2 months ago
  1. The 'Skip' button is displayed on two lines. image
  2. The 'Receive' function in 'send and receive' isn't working on certain devices, including some Pixel, Xiaomi, Vivo, Oppo, HUAWEI, and Samsung models. https://github.com/user-attachments/assets/0eb88511-826c-4a3a-aafd-5ca57a54082b
  3. Tweets would need updates (tracking this here for now) image

Sharing screenshot for #1 in a bit UPDATE: Please ignore #1, i was in a different version of the app. On the latest version it works!! @reasje

cc: @sheenhx

reasje commented 2 months ago

@SanghamitraBhowmick1993 Thanks for reporting

  1. will be fixed
  2. Strange, Can you plz list the models, So that I can have close look? (pixel 6 does not work and 7 works on android 13)
reasje commented 2 months ago

@SanghamitraBhowmick1993 Just pushed

  1. Should be fixed
  2. Should be fixed. (In other languages than English)
SanghamitraBhowmick1993 commented 2 months ago

3 works, tested in pixel 7

image

4 tested in DE and works, @sheenhx would you also please help confirm if it works for you for #4

image

cc: @reasje

SanghamitraBhowmick1993 commented 2 months ago

5 Please help disable the light theme @reasje . This has been confirmed and aligned with @sheenhx

image