search

bluerobotics / companion

Companion computer startup scripts and examples
https://www.ardusub.com/operators-manual/companion-web.html
GNU General Public License v3.0
48 stars 54 forks source link

post-update: Add reboot after update #308

Closed patrickelectric closed 4 years ago

patrickelectric commented 4 years ago

Fix #123

Signed-off-by: Patrick José Pereira patrickelectric@gmail.com

Williangalvani commented 4 years ago

@jaxxzer mentioned that this blanks the page. We should check if it still happens and fix it if so. My guess is that it is due to the ajax in the background or open websockets (we'd just have to stop them)

patrickelectric commented 4 years ago

@Williangalvani I did check, and this does not happen with this PR. That's why I started the reboot process as detached to avoid such possible problem.

patrickelectric commented 4 years ago

@Williangalvani did you test the PR ?

Williangalvani commented 4 years ago

@Williangalvani I did check, and this does not happen with this PR. That's why I started the reboot process as detached to avoid such possible problem.

sounds good, then.

patrickelectric commented 4 years ago

deepin-screen-recorder_Select area_20200130123708

jaxxzer commented 4 years ago

wait for me to test and I'll merge @patrickelectric