Skip to content

Conversation

@nvrakesh06
Copy link
Contributor

TICKET: WIN-6658

hitansh-madan
hitansh-madan previously approved these changes Aug 20, 2025
}

// Validate wallet type if coin requires it
if (teConfig.validateWallet && this._wallet.type) {
Copy link
Contributor

Choose a reason for hiding this comment

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

nit:

Suggested change
if (teConfig.validateWallet && this._wallet.type) {
if (typeof teConfig.validateWallet === 'function' && this._wallet.type) {

@nvrakesh06 nvrakesh06 marked this pull request as ready for review August 25, 2025 15:21
@nvrakesh06 nvrakesh06 requested review from a team as code owners August 25, 2025 15:21
Copy link
Contributor

@venkateshv1266 venkateshv1266 left a comment

Choose a reason for hiding this comment

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

LGTM

@nvrakesh06 nvrakesh06 merged commit 6d00970 into master Aug 25, 2025
12 checks passed
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.

4 participants