Differential D15953
[Cashtab] Better switch mgmt in send token Authored by bytesofman on Thu, Apr 11, 23:05.
Details
T3528 For send token, we only want one token feature enabled at a time. We will be adding more token features for different token types, and it is already a bit of a mess to manually turn off "all the other switches" every time a switch comes on. Standardize the approach so it will be easier to add NFT features. npm test
Diff Detail
Event TimelineComment Actions This is already the behavior in prod isn't it? Toggling another send token function will disable all others. Comment Actions Yes -- but it is implemented badly. Adding any switch means you need to update the handleToggle method of every other switch. Before I start adding switches for NFTs, want this to be easier to manage. After this diff, they all take the same method. The added test is confirming we keep expected behavior. |