Skip to content

feat(base): loadMarketsAndSignIn#27622

Merged
carlosmiei merged 1 commit intoccxt:masterfrom
ttodua:loadWithSignIn
Jan 5, 2026
Merged

feat(base): loadMarketsAndSignIn#27622
carlosmiei merged 1 commit intoccxt:masterfrom
ttodua:loadWithSignIn

Conversation

@ttodua
Copy link
Member

@ttodua ttodua commented Jan 5, 2026

there are many cases of non-optimal flow like in all private methods (related exchanges):

async fetchXyz () {
   await this.loadMarkets();
   await this.signIn ();

}

so we would call just loadMarketsAndSignIn

@carlosmiei
Copy link
Collaborator

carlosmiei commented Jan 5, 2026

@ttodua We can add it but be careful, when for instance we need to sign in to call fetchCurrencies or any other private endpoint because the order is not assured

@carlosmiei carlosmiei merged commit b11b5ef into ccxt:master Jan 5, 2026
6 of 11 checks passed
@carlosmiei carlosmiei deleted the loadWithSignIn branch January 5, 2026 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants