MXCzkEVM / datadash-wallet

Other
0 stars 1 forks source link

DApps image caching #185

Open SanghamitraBhowmick1993 opened 2 months ago

SanghamitraBhowmick1993 commented 2 months ago

https://www.notion.so/wannsee-roadmap/DApps-image-caching-e3da5e6d125e47d8a1bf6fdb1e68cf1e

reasje commented 2 months ago

I want to include the json files of data which are (Tokens list & DApps) as sub module to AXS repo, Which will help getting updates instantly. Then I can use their json inside app as assets whenever the Github is not responding for a particular use.

reasje commented 2 months ago

I have been able to initialize dapp store as submodule to Shared Repo and also use It's content to load data initially If anyhing goes wrong with getting data from Github. I will be debuging It and once everything is fine I will move to token list local data. Other data needs to be made local: IPFS - Tweets

reasje commented 2 months ago

Token list local data has been initialized and works fine I am moving to work on IPFS - Tweets

reasje commented 2 months ago

Some improvements are done down the road in order to imprv project structure:

  1. Separating Config from Utils
  2. Adding more helpers to reduce code in Config & Utils
  3. Some imports are updated systematically in order to reduce the boilerplate imports and duplications
reasje commented 2 months ago

Flutter analyzer is terminated, This prevents me from developing, This happened before but this time It doesn't work at all, I am checking for solution.

reasje commented 2 months ago

I was abel to solve the issue, Not sure what exactly was the fix, Because changes effecetd after sometime:

  1. Github guidence
  2. Changing some imports in contracts, The file that was mentioned to be missing analyzing
reasje commented 2 months ago

I have finished IPFS & tweets local data impl, I will start debugging.

reasje commented 2 months ago

@SanghamitraBhowmick1993 Debugging is finished, I have merged the changes to pre_main_qa. For testing purpose : If Github works fine in your region plz turn off your internet and open application, YOu should be able to see : DApps with their images - Tokens list with their images - tweets card loading (Since loading from Tweeter needs connection) - IPFS gateway won't show in chain configuration settings because we need to test every ipfs gateway (which requires connection) in order to choose one.

SanghamitraBhowmick1993 commented 2 months ago

Tweets does not load w/o connection, but the dapp with their images/token list loads and working as expected. Could we move this and the permission exception changes to main @reasje so that we can raise a new playstore request as confirmed/aligned with @sheenhx

reasje commented 2 months ago

@SanghamitraBhowmick1993 Thanks for detailed information, If you see tweets cards without internet then It mean It's loaded from local, If the tweets are not showing It's expected because the connection is required to get tweets. I have just merged to main.