Support redemptions of Firma for $1 of XEC
Details
- Reviewers
emack - Group Reviewers
Restricted Project - Commits
- rABCbe5be3147e03: [Cashtab] Support one-click redemptions
npm test
Do it live at the test site, https://cashtab-local-dev.netlify.app/#/token/0387947fd575db4fb19a3e322f635dec37fd192b5941625b66bc4b2c3008cbf0
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Since the button will only work based on the external api, it's better to not show the button if the api is unavailable for some reason.
I suggest you do this:
- Attempt to retrieve the price from that endpoint
- If not successful, don't show the button. Otherwise show the button with the actual price, like "Redeem FIRMA for 1234 XEC"
- Not sure if it's worth refreshing periodically (in the even the price changes quickly)
Another point: can the slider be adjusted so that it starts at the minimum quantity and not at zero ? So you cannot move the cursor to the invalid zone, better UX
Consider a 100% 50% 25% button above the Redeem button similar to binance. Easier for tax calculations when people are just partially selling rounded percentages.
cashtab/src/components/Etokens/Token/index.tsx | ||
---|---|---|
2377 ↗ | (On Diff #52772) | For the "Actual price", perhaps update to x,xxx XEC / FIRMA, otherwise it reads as if you'll be receiving that approximate amount selling a fraction of a FIRMA. |
yes, will need to get this in a separate diff. this is a generalized thing we need to put on the slider.
cashtab/src/components/Etokens/Token/index.tsx | ||
---|---|---|
2377 ↗ | (On Diff #52772) | added a "you receive" line that shows total amount |
I no longer see the redeem button anymore even though I have a FIRMA balance and am on the FIRMA token landing page.
v3.14.0 on the netlify instance after multiple refreshes
it's back up now. I had pushed a different version to test the migration related to D17650
As per tg, noting the "minor price change in between clicking list and the listing getting published" issue that is for another diff, perhaps a relist on latest price button or some sort.