Fara Woolf
Fara Woolf
@kyranjamie should I rebase this and take it over to see if helps the fee loading issue? Our fee row seems really broken atm.
Sorry, I prob just really created a lot of conflicts for you with my last merge. If helps, I can rebase this for you?
Related? https://github.com/leather-wallet/extension/issues/4926
@kyranjamie starting work on this, so let me know any other thots/prefs you have here atm. In adding other tokens, we've been using the data structure of: ```ts const queriedTokenData...
I suppose to adhere to clean code, using `tokenData` isn't great. But, then you typ get `token.token...` later, so prob going back to `asset` is better here? 🤔
Maybe it isn't nec to combine them? I know we discussed at first whether or not to combine them into one data model ...thinking maybe we should separate them as...
This was fixed in: https://github.com/leather-io/extension/pull/5571
Adding link to original PR here for ref: https://github.com/leather-wallet/extension/pull/4958
The status of this issue is `Closed` so closing the issue.
@markmhendrickson it might be that we want to prioritize this refactoring now that we have move fiat values getting pushed?