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

chore: fix some typos #6348

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/backup/BackupQuiz.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ const TAG = 'backup/BackupQuiz'

const MNEMONIC_BUTTONS_TO_DISPLAY = 6

// miliseconds to wait until showing success or failure
// milliseconds to wait until showing success or failure
const CHECKING_DURATION = 1.8 * 1000

export enum Mode {
Expand Down
6 changes: 3 additions & 3 deletions src/fiatconnect/ReviewScreen.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ const usePrepareBaseFiatConnectOutTransactions = ({
)
}

// note that the receipient address on the prepared transaction is the
// note that the recipient address on the prepared transaction is the
// user's own address. this will be replaced by the fiatconnect provider
// transfer address in the sagas if the user proceeds with the transaction.
// the provider address is not known until making a POST request to the
Expand Down Expand Up @@ -323,8 +323,8 @@ export default function FiatConnectReviewScreen({ route, navigation }: Props) {
serializablePreparedTransaction:
prepareTransactionsResult.result?.type === 'possible'
? getSerializablePreparedTransaction(
prepareTransactionsResult.result.transactions[0] // there should be only one transaction
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@crazeteam, you have a lint error here; mind running yarn lint in the repository's root and committing the changes.

)
prepareTransactionsResult.result.transactions[0] // there should be only one transaction
)
: undefined,
networkId: token?.networkId,
})
Expand Down
2 changes: 1 addition & 1 deletion src/redux/sagas.ts
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ function* loggerSaga() {
) {
// Log only action type, but not the payload as it can have sensitive
// information or information that is not helpful for debugging. Excluding
// all IDENTITY/ actions because high likelyhood they contain PII and the
// all IDENTITY/ actions because high likelihood they contain PII and the
// blocklist may get out of date.
Logger.debug('redux/saga@logger', `${action.type} (payload not logged)`)
return
Expand Down
2 changes: 1 addition & 1 deletion src/utils/time.ts
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ function timeDifference(currTime: number, prevTime: number) {
return 1.0 * (millisecondsSinceEpoch(currTime) - millisecondsSinceEpoch(prevTime))
}

// some timestamps are in seconds, some are in miliseconds
// some timestamps are in seconds, some are in milliseconds
// assume dates will be within a few decades of now and multiple accordingly
function millisecondsSinceEpoch(timestamp: number) {
return Math.abs(differenceInYears(timestamp, Date.now())) > 40 ? timestamp * 1000 : timestamp
Expand Down
Loading