Bug: Broker page, Update Broker should not be enabled without generating the URL

theheapdump / kite-webapp

web-app for kite trading-bot platform

0 stars 1 forks source link

Closed bevanamit closed 3 years ago

bevanamit commented 3 years ago

theheapdump commented 3 years ago

@kshitijbhardwaj - pls take care

bevanamit commented 3 years ago

@Anmol: Is it possible to verify that the Client has put proper inputs for broker registration ? @anmoldeep0123

theheapdump commented 3 years ago

@bevanamit @kshitijbhardwaj zerodha client will not be other then XA1346 -

2 alphabets + 4 alphanumeric

secret and apikey - we have to trust the user

generate URL is static content

theheapdump commented 3 years ago

zerdoha does not allow typing more than 6 in client ID

theheapdump commented 3 years ago

@bevanamit @kshitijbhardwaj

bevanamit commented 3 years ago

tested