multivisor-cli /api url fix

tiagocoutinho / multivisor

Centralized supervisor WebUI and CLI

GNU General Public License v3.0

189 stars 35 forks source link

multivisor-cli /api url fix #36

Closed guy881 closed 5 years ago

guy881 commented 5 years ago

I haven't tried multivisor-cli until now and I found it's not working. :( I realized it's because of my change to urls introduced not long ago. This PR fixes this problem.

@tiagocoutinho I am asking you for a review.

tiagocoutinho commented 5 years ago

Great! Many thanks. My fault for not having setup unit tests yet!

LGTM. Merging.

guy881 commented 5 years ago

Thanks, @tiagocoutinho! Unit tests will be definitely useful, I will open an issue about this. ;)