Giveth / giveth-dapp

Giveth Dapp for crowdfunding and managing donations on the ethereum blockchain.
https://beta.giveth.io
GNU General Public License v3.0
358 stars 230 forks source link

Sadly we were unable to get the exchange rate! #2543

Open RamRamez opened 2 years ago

RamRamez commented 2 years ago

Always getting this error on develop, but it's ok on trace.giveth.io

image

RamRamez commented 2 years ago

In trace.giveth.io it's ok, but when I use master branch with develop backend, I get same error. It seems it's related to backend. @mohammadranjbarz Can you check it?

mohammadranjbarz commented 2 years ago

Always getting this error on develop, but it's ok on trace.giveth.io

This issue is also a blocker for issue #2544

image

@RamRamez Are you getting this error now? I can't reproduce it

RamRamez commented 2 years ago

@mohammadranjbarz Yes I'm still getting it. My profile token is DAI and convertMultipleRates doesn't give current balance value of Trace in DAI

image

mohammadranjbarz commented 2 years ago

@mohammadranjbarz Yes I'm still getting it. My profile token is DAI and convertMultipleRates doesn't give current balance value of Trace in DAI

image

@RamRamez Ok Gotcha, the DAI is not in token whitelist in develop , so you get error, in develop environment you should not allow user to select DAI user can select XDAI please see our token whitelist in develop https://feathers.develop.giveth.io/whitelist

Screen Shot 1400-07-05 at 11 07 12

but it's not good that we don't return the appropriate error message on the backend

mohammadranjbarz commented 2 years ago

@mohammadranjbarz Yes I'm still getting it. My profile token is DAI and convertMultipleRates doesn't give current balance value of Trace in DAI image

@RamRamez Ok Gotcha, the DAI is not in token whitelist in develop , so you get error, in develop environment you should not allow user to select DAI user can select XDAI please see our token whitelist in develop https://feathers.develop.giveth.io/whitelist

Screen Shot 1400-07-05 at 11 07 12

but it's not good that we don't return the appropriate error message on the backend

@RamRamez Sorry my mistake., I think it's a bug in the backend, I should have return results for native currency list