Skip to content

Commit

Permalink
fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
tyleroooo committed Jan 24, 2025
1 parent 5d66100 commit d10d00a
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 9 deletions.
3 changes: 3 additions & 0 deletions src/constants/markets.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,9 @@ export type MarketData = {
// Displayable Market id (e.g. 'ETH-USD' or 'BUFFI-USD')
displayId: Nullable<string>;

// Displayable asset id so just 'ETH' or 'BUFFI
displayableAsset: Nullable<string>;

clobPairId: string;
effectiveInitialMarginFraction: Nullable<number>;
logo: string;
Expand Down
10 changes: 9 additions & 1 deletion src/lib/marketsHelpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,11 @@ import { AssetData, PerpetualMarketSummary } from '@/abacus-ts/types/summaryType

import { ISOLATED_LIQUIDITY_TIER_INFO, MarketData } from '@/constants/markets';

import { getDisplayableTickerFromMarket, getMarketIdFromAsset } from './assetUtils';
import {
getDisplayableAssetFromTicker,
getDisplayableTickerFromMarket,
getMarketIdFromAsset,
} from './assetUtils';
import { BIG_NUMBERS, MustBigNumber } from './numbers';
import { safeAssign } from './objectHelpers';

Expand All @@ -28,6 +32,7 @@ export function getMarketDataFromPerpetualMarketSummary(
ticker: id,
assetId,
displayableTicker: displayId,
displayableAsset,
clobPairId,
effectiveInitialMarginFraction,
logo,
Expand All @@ -54,6 +59,7 @@ export function getMarketDataFromPerpetualMarketSummary(
id,
assetId,
displayId,
displayableAsset,
clobPairId,
effectiveInitialMarginFraction,
logo,
Expand Down Expand Up @@ -94,11 +100,13 @@ export function getMarketDataFromAsset(asset: AssetData, favoritedMarkets: strin

const ticker = getMarketIdFromAsset(assetId);
const displayId = getDisplayableTickerFromMarket(ticker);
const displayableAsset = getDisplayableAssetFromTicker(ticker);

return safeAssign({}, {
id: ticker,
assetId,
displayId,
displayableAsset,
clobPairId: '-1',
effectiveInitialMarginFraction: ISOLATED_LIQUIDITY_TIER_INFO.initialMarginFraction,
logo,
Expand Down
10 changes: 4 additions & 6 deletions src/lib/orders.ts
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ export const isMarketOrderTypeNew = (type?: IndexerOrderType) =>
IndexerOrderType.STOPMARKET,
IndexerOrderType.TAKEPROFITMARKET,
IndexerOrderType.TRAILINGSTOP,
].some((t) => t === type);
].includes(type);

export const isLimitOrderType = (type?: AbacusOrderTypes) =>
type &&
Expand All @@ -180,9 +180,7 @@ export const isLimitOrderType = (type?: AbacusOrderTypes) =>

export const isLimitOrderTypeNew = (type?: IndexerOrderType) =>
type &&
[IndexerOrderType.LIMIT, IndexerOrderType.STOPLIMIT, IndexerOrderType.TAKEPROFIT].some(
(t) => t === type
);
[IndexerOrderType.LIMIT, IndexerOrderType.STOPLIMIT, IndexerOrderType.TAKEPROFIT].includes(type);

export const isStopLossOrder = (order: SubaccountOrder, isSlTpLimitOrdersEnabled: boolean) => {
const validOrderTypes = isSlTpLimitOrdersEnabled
Expand All @@ -197,7 +195,7 @@ export const isStopLossOrderNew = (
const validOrderTypes = isSlTpLimitOrdersEnabled
? [IndexerOrderType.STOPLIMIT, IndexerOrderType.STOPMARKET]
: [IndexerOrderType.STOPMARKET];
return validOrderTypes.some((type) => type === order.type) && order.reduceOnly;
return order.reduceOnly && validOrderTypes.includes(order.type);
};

export const isTakeProfitOrder = (order: SubaccountOrder, isSlTpLimitOrdersEnabled: boolean) => {
Expand All @@ -214,7 +212,7 @@ export const isTakeProfitOrderNew = (
const validOrderTypes = isSlTpLimitOrdersEnabled
? [IndexerOrderType.TAKEPROFIT, IndexerOrderType.TAKEPROFITMARKET]
: [IndexerOrderType.TAKEPROFITMARKET];
return validOrderTypes.some((type) => type === order.type) && order.reduceOnly;
return order.reduceOnly && validOrderTypes.includes(order.type);
};

export const isSellOrder = (order: SubaccountOrder) => {
Expand Down
4 changes: 2 additions & 2 deletions src/views/menus/useGlobalCommands.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -91,11 +91,11 @@ export const useGlobalCommands = (): MenuConfig<string | number, string | number
{
group: 'markets',
groupLabel: stringGetter({ key: STRING_KEYS.MARKETS }),
items: markets.map(({ name, id, logo, displayId, assetId }) => ({
items: markets.map(({ name, id, logo, displayableAsset, assetId }) => ({
value: id,
slotBefore: <AssetIcon logoUrl={logo} symbol={assetId} />,
label: name,
tag: displayId,
tag: displayableAsset,
onSelect: () => navigate(`${AppRoute.Trade}/${id}`),
})),
},
Expand Down

0 comments on commit d10d00a

Please sign in to comment.