refactor(sb): price movement type

blockchain · Jul 15, 2020 · 96f8041 · 96f8041
1 parent c2ba940
commit 96f8041
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/...t-v4-frontend/src/modals/SimpleBuy/CryptoSelection/CryptoSelector/PriceMovement/index.tsx b/...t-v4-frontend/src/modals/SimpleBuy/CryptoSelection/CryptoSelector/PriceMovement/index.tsx
@@ -1,7 +1,7 @@
 import { actions } from 'data'
 import { bindActionCreators, Dispatch } from 'redux'
 import { connect, ConnectedProps } from 'react-redux'
-import { FiatType, SBPairType } from 'core/types'
+import { FiatType, PriceMovementDirType, SBPairType } from 'core/types'
 import { getCoinFromPair } from 'data/components/simpleBuy/model'
 import { RootState } from 'data/rootReducer'
 import React, { PureComponent } from 'react'
@@ -18,7 +18,7 @@ const Change = styled.span<{ color: keyof DefaultTheme }>`
   color: ${props => props.theme[props.color]};
 `
 
-const getSignFromMovement = (movement: 'none' | 'up' | 'down') => {
+const getSignFromMovement = (movement: PriceMovementDirType) => {
   switch (movement) {
     case 'down':
       return '-'
@@ -29,7 +29,7 @@ const getSignFromMovement = (movement: 'none' | 'up' | 'down') => {
   }
 }
 
-const getColorFromMovement = (movement: 'none' | 'up' | 'down') => {
+const getColorFromMovement = (movement: PriceMovementDirType) => {
   switch (movement) {
     case 'down':
       return 'red600'

diff --git a/packages/blockchain-wallet-v4/src/redux/data/misc/actions.ts b/packages/blockchain-wallet-v4/src/redux/data/misc/actions.ts
@@ -1,5 +1,10 @@
 import * as AT from './actionTypes'
-import { CoinType, FiatType, MiscActionTypes } from 'core/types'
+import {
+  CoinType,
+  FiatType,
+  MiscActionTypes,
+  PriceMovementDirType
+} from 'core/types'
 
 // FETCH_CAPTCHA
 export const fetchCaptcha = () => ({ type: AT.FETCH_CAPTCHA })
@@ -25,7 +30,7 @@ export const fetchPrice24HLoading = (base: CoinType): MiscActionTypes => ({
 export const fetchPrice24HSuccess = (
   base: CoinType,
   change: string,
-  movement: 'none' | 'up' | 'down'
+  movement: PriceMovementDirType
 ): MiscActionTypes => ({
   type: AT.FETCH_PRICE_24H_SUCCESS,
   payload: {

diff --git a/packages/blockchain-wallet-v4/src/redux/data/misc/types.ts b/packages/blockchain-wallet-v4/src/redux/data/misc/types.ts
@@ -14,6 +14,8 @@ export type RatesType = {
   [key in FiatType]: RateType
 }
 
+export type PriceMovementDirType = 'none' | 'up' | 'down'
+
 // state
 export type MiscStateType = {
   authorize_login: RemoteDataType<any, any>
@@ -24,7 +26,7 @@ export type MiscStateType = {
   price_24h: {
     [key in CoinType]: RemoteDataType<
       string,
-      { change: string; movement: 'none' | 'up' | 'down' }
+      { change: string; movement: PriceMovementDirType }
     >
   }
   price_index_series: RemoteDataType<any, any>
@@ -89,7 +91,7 @@ interface FetchPrice24HSuccessActionType {
   payload: {
     base: CoinType
     change: string
-    movement: 'none' | 'up' | 'down'
+    movement: PriceMovementDirType
   }
   type: typeof AT.FETCH_PRICE_24H_SUCCESS
 }