Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds transfer status indicator to deposit withdraw #1023

Merged
merged 70 commits into from
Nov 25, 2022

Conversation

milesz8
Copy link
Contributor

@milesz8 milesz8 commented Nov 19, 2022

#789

When a channel for a deposit or withdraw is blocked or congested the button will change color and note the state of the channel.
Screenshot from 2022-11-20 16-23-11

@vercel
Copy link

vercel bot commented Nov 19, 2022

@milesz8 is attempting to deploy a commit to the OsmoLabs Team on Vercel.

A member of the Team first needs to authorize it.

packages/stores/types/queries-external/ibc/types.d.ts Outdated Show resolved Hide resolved
packages/stores/src/queries-external/ibc/status-by-id.ts Outdated Show resolved Hide resolved
packages/stores/src/queries-external/ibc/status-by-id.ts Outdated Show resolved Hide resolved
packages/stores/src/queries-external/ibc/status-by-id.ts Outdated Show resolved Hide resolved
packages/stores/src/queries-external/ibc/status-by-id.ts Outdated Show resolved Hide resolved
packages/web/components/trade-clipboard/index.tsx Outdated Show resolved Hide resolved
packages/web/modals/ibc-transfer.tsx Outdated Show resolved Hide resolved
packages/web/modals/ibc-transfer.tsx Outdated Show resolved Hide resolved
packages/web/modals/ibc-transfer.tsx Outdated Show resolved Hide resolved
packages/web/localizations/en.json Outdated Show resolved Hide resolved
JeremyParish69 and others added 22 commits November 22, 2022 13:45
…und-img-circle

Replace FUND token logo with Circular logo
Unverified because Marble no longer receives incentives.
* Fix commands in README.md

* Remove arm64 support

* Pin node version to 18.12.1

* Remove duplicate Dockerfile and docker-compose.yml
* Changed Marble to unverified

Unverified because Marble no longer receives incentives.

* Changed IXO to unverified

Unverified because IXO no longer receives incentives.
@vercel
Copy link

vercel bot commented Nov 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
osmosis-frontend ❌ Failed (Inspect) Nov 25, 2022 at 9:42PM (UTC)
osmosis-frontier ✅ Ready (Inspect) Visit Preview Nov 25, 2022 at 9:42PM (UTC)

@jonator
Copy link
Member

jonator commented Nov 25, 2022

Thanks 🔥🚀

@jonator jonator merged commit 0e78486 into osmosis-labs:master Nov 25, 2022
@CryptoAssassin1
Copy link
Member

readonly getIbcStatus = computedFn(
(channelId: string): IbcStatus | undefined => {
const channelData = this.response?.data.find(
(channel) => channel.channel_id === channelId
);
if (channelData) {
if (channelData.size_queue > 0) {
if (channelData.duration_minutes > 20) return "blocked";
else if (channelData.duration_minutes > 5) return "congested";
}
return "normal";
}
}
I think (channelData.size_queue > 0) should be 10 or more then 10

JoseRFelix pushed a commit that referenced this pull request Dec 5, 2022
* Add ALTER, SHD & SIENNA (Secret SNIP-20 tokens)

* Update SIENNA's image

* Update ALTER's image

* Add Button group token

* added SHD, BUTT, & ALTER svgs

* added staking derivatives

* added ics20 contract to ibc-assets

* updated to channel-476

* updated ics20 contract

* removed accidental changes

* fix comment

* stkd-SCRT

* cleanup

* Update ibc-assets.ts

* Adds transfer status indicator to deposit withdraw

* Revert some unneeded changes to package.json files

* Removes type that was put directly in observable query

* Fix content overflow on zh-tw (#1024)

* Fix #1011 Non-localized stablecoin type (#1025)

* add sample fix for ko + es

* use collateral type as key into localization

* fix key for install keplr notice

* Remove unneeded type definition

* uses implied return undefined and spelling

* Remove unused store import

* Remove default baseUrl, Add in channel status logic

* Remove unused import and use deconstructed values directly

* Remove log

* Update button text for blocked and congested

* Switch to CoinGecko IDs--no pool spot prices (#1026)

* Fix content overflow on zh-tw (#1028)

* Fix content overflow on zh-tw

* Fix content overflow on languages with Unicode/full width chars

* Add missing entry

* correct stargaze cgid, uncomment prices

* add ibc-go to chihuahua

* add secret network features and change api

* minor fix

* add secret network tokens

* Use `computedFn`

* Don't get prices when no input in swap tool

* Enable pool calculation

* update korean.json (#1037)

* Uncomment image

* removed duplicated secret network tokens

* Update Chihuahua Chain Suggest Fees

* Replace FUND token logo with Circular logo

Was a square before.

* Change crbruc to unstable, unverified (#1040)

* Fix pool reward query error response

* Changed Marble to unverified (#1045)

Unverified because Marble no longer receives incentives.

* Fix zh-tw format (#1043)

* Fix Docker GitHub Actions  (#1044)

* Fix commands in README.md

* Remove arm64 support

* Pin node version to 18.12.1

* Remove duplicate Dockerfile and docker-compose.yml

* Make NETA, BOOT unverified

* Changed IXO to unverified (#1049)

* Changed Marble to unverified

Unverified because Marble no longer receives incentives.

* Changed IXO to unverified

Unverified because IXO no longer receives incentives.

* Adds transfer status indicator to deposit withdraw

* Revert some unneeded changes to package.json files

* Removes type that was put directly in observable query

* Remove unneeded type definition

* uses implied return undefined and spelling

* Remove unused store import

* Remove default baseUrl, Add in channel status logic

* Remove unused import and use deconstructed values directly

* Remove log

* Update button text for blocked and congested

Co-authored-by: Assaf Morami <assaf.morami@gmail.com>
Co-authored-by: DrPresident <jacksonswenson@securesecrets.org>
Co-authored-by: PikachuEXE <pikachuexe@gmail.com>
Co-authored-by: Sehyun Chung ✌︎ <sehyun@berkeley.edu>
Co-authored-by: JeremyParish69 <95667791+JeremyParish69@users.noreply.github.com>
Co-authored-by: Jeremy Parish <jeremyparish69@gmail.com>
Co-authored-by: jonator <jonathanator0@gmail.com>
Co-authored-by: KIM, WOOJUNG <gnujoow@users.noreply.github.com>
Co-authored-by: Crypto Assassin <103904125+CryptoAssassin1@users.noreply.github.com>
Co-authored-by: Niccolo Raspa <6024049+niccoloraspa@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants