Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

AppStore screen error during Server upgrade. #1737

Closed
panaaj opened this issue May 17, 2024 · 2 comments · Fixed by #1738
Closed

AppStore screen error during Server upgrade. #1737

panaaj opened this issue May 17, 2024 · 2 comments · Fixed by #1738
Labels
bug

Comments

@panaaj
Copy link
Member

panaaj commented May 17, 2024 •
edited

When upgrading server version (2.7.2 -> 2.8.0) the AppStore screen shows the following error...

image

The Browser console shows the following error:

TypeError: Cannot set properties of undefined (setting 'installing')
    at Apps.js:37:35
    at Array.forEach (<anonymous>)
    at Apps.js:36:31
    at Apps.js:87:19
    at Gi (react-dom.production.min.js:153:146)
    at To (react-dom.production.min.js:261:496)
    at Eu (react-dom.production.min.js:246:265)
    at xu (react-dom.production.min.js:246:194)
    at mu (react-dom.production.min.js:239:172)
    at react-dom.production.min.js:123:115

The Server -> Update screen shows the status of the install.

After server has been restarted the AppStore is functional again.
@panaaj panaaj added the bug label May 17, 2024
@cmotelet
Copy link
Member

If you don't restart the server and look at the processes, the update is launched anyway:
May 17 12:18:57 barco signalk-server[677587]: POST /skServer/appstore/install/signalk-server/2.8.0 200 258.889 ms - 28
May 17 12:18:57 barco sudo[677637]: pi : PWD=/home/pi/.signalk ; USER=root ; COMMAND=/usr/bin/npm install -g signalk-server@2.8.0
The user is not informed of the update in progress, but a little later at the end of the update:
image
After restarting the server everything works in 2.8.0

tkurki added a commit that referenced this issue May 17, 2024
@tkurki
fix: error in admin ui when updating server
7f4db3a 
Fixes #1737.
@tkurki tkurki mentioned this issue May 17, 2024
Merged
@tkurki
Copy link
Member

tkurki commented May 17, 2024

Yeah, just a ui glitch. it would help if somebody couild test the fix in the linked PR, i think it solves the immediately problem but there may be more related to the App store revamp & server updates, that is happening now for the first time.

tkurki added a commit that referenced this issue May 18, 2024
@tkurki
fix: error in admin ui when updating server
f1e7d91 
Fixes #1737.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: error in admin ui when updating server SignalK/signalk-server
3 participants
@tkurki @cmotelet @panaaj