fix: fixes issue with migrating category settings to the server

[ErikBjare]

Fixes ActivityWatch/activitywatch#1067

Analysis of the issue in ActivityWatch/activitywatch#1067 (comment)

[codecov]

Codecov Report

Attention: Patch coverage is 9.09091% with 10 lines in your changes missing coverage. Please review.

Project coverage is 26.13%. Comparing base (021acc4) to head (07bcda8).
Report is 3 commits behind head on master.

Files	Patch %	Lines
src/stores/settings.ts	9.09%	10 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #583      +/-   ##
==========================================
- Coverage   26.15%   26.13%   -0.02%     
==========================================
  Files          27       27              
  Lines        1629     1630       +1     
  Branches      285      273      -12     
==========================================
  Hits          426      426              
- Misses       1177     1178       +1     
  Partials       26       26              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[BelKed]

Manually verified, works as intended. The error doesn't show up in the UI and the data is displayed normally :)

[ErikBjare]

@BelKed Did you check the codepath by reproducing the error? (possibly by removing the entry in the server-side settings)

[BelKed]

@BelKed Did you check the codepath by reproducing the error? (possibly by removing the entry in the server-side settings)

Yes, exactly:

1. Removed the classes entry from the database.
2. Added an empty classes entry to the localStorage (localStorage.setItem("classes", ""))
3. The error doesn't appear anymore :)

[ErikBjare]

Awesome, triggering a build for v0.13.1 now.