balta2ar / brotab

Control your browser's tabs from the command line
MIT License
406 stars 29 forks source link

API replies "OK" when sending an incorrect tab move request #42

Open CRImier opened 4 years ago

CRImier commented 4 years ago

When sending a tab move request with the wrong (non-recalculated) tab ID, the API will reply "OK" and will not do anything (no tabs will be moved). I've solved it in my script by using the brotab API internal tab rindex ecalculation logic, and I've read the comments in the code that say you want to keep the recalculation logic in extensions - that's fine, but it will be good to have the API respond with something other than "OK" when a move action fails.