Skip to content

Commit

Permalink
Merge pull request #261 from zhoufanjin/master
Browse files Browse the repository at this point in the history
chore: fix some comments
  • Loading branch information
vrtnd authored Aug 6, 2024
2 parents 6450bbd + d92d2c1 commit 71d7849
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/adapters/orbitbridge/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ Docs: https://bridge-docs.orbitchain.io/
Adapter is finished for chains with a vault: Ethereum, BSC, (also on Klaytn).
Issue is that other chains have minter contracts, and there appears to be multiple (at least on Polygon).
It's difficult to find them all, but can use explorer txs to find them: https://scope.klaytn.com/bridge
(stil not sure I have found them all, some chains do not have much activity).
(still not sure I have found them all, some chains do not have much activity).
Seems like Klaytn has both vault and minters, not sure if this is also true for Ethereum or BSC.
Expand Down
2 changes: 1 addition & 1 deletion src/adapters/portal/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ const portalNativeAndWrappedTransfersFromHashes = async (chain: Chain, hashes: s
return aggregated;
};

// checks deposits for Solana as receipient chain, withdrawals for Solana as source chain and returns only those txs
// checks deposits for Solana as recipient chain, withdrawals for Solana as source chain and returns only those txs
// the 'wormhole chain id' for Solana is 1
const processLogsForSolana = async (logs: EventData[], chain: Chain) => {
const provider = getProvider(chain) as any;
Expand Down
2 changes: 1 addition & 1 deletion src/utils/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export type EventData = {
chainOverride?: string; // used to insert tx using bridgeID from same bridgeNetwork but a different chain
isUSDVolume?: boolean; // used to insert tx without specifying any token, only a USD value
txsCountedAs?: number; // used to insert tx and have it count as multiple txs (only affects transaction counts in hourly/daily aggregated entries)
timestamp?: number; // timetamp of the block if provided
timestamp?: number; // timestamp of the block if provided
};

export type EventKeyMapping = {
Expand Down
2 changes: 1 addition & 1 deletion src/utils/wrappa/postgres/write.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ export const insertTransactionRow = async (
},
onConflict: "ignore" | "error" | "upsert" = "error"
) => {
// FIX should use dynamicly built strings here, I just didn't finish it
// FIX should use dynamically built strings here, I just didn't finish it
let sqlCommand = sql`
insert into bridges.transactions ${sql(params)}
`;
Expand Down

0 comments on commit 71d7849

Please sign in to comment.