Cleanup done #878

Merged

a22erigr merged 1 commit from #869-tempViews-cleanup into team_2_week_5

2026-05-11 11:48:43 +00:00

c22abdal commented

2026-05-11 09:55:07 +00:00

Collaborator

Due to some local issue, I couldn't verify that the decimals have been reduced to one.
However, code-wise, the change has been made in TemperatureView.js

Due to some local issue, I couldn't verify that the decimals have been reduced to one. However, code-wise, the change has been made in TemperatureView.js

c22abdal added the

needs review

label

2026-05-11 09:55:07 +00:00

c22abdal added 1 commit

2026-05-11 09:55:08 +00:00

Cleanup done 0addc3bddf

Due to some local issue, i couldn't verify that the decimals have been reduced to one. However, code wise the change has been made in TemperatureView.js

c24danli requested review from j21marbr

2026-05-11 10:52:10 +00:00

j21marbr commented

2026-05-11 11:22:01 +00:00

Collaborator

Review on #878

Whats done

Tested on macOS using Chrome

App starts
- Verified that the SMHI forecast now uses Swedish date formatting
- Verified that the temperature now shows one decimal instead of two
- No issues found after the SQLite package version update

Inspected following files

SMHIForecastView.js
TemperatureView.js
energySensorMockup.csproj

What needs to be done

No issues found.

Conclusion

Works as intended and the code is ready to be merged.

# Review on #878 ## Whats done ### Tested on macOS using Chrome + App starts + Verified that the SMHI forecast now uses Swedish date formatting + Verified that the temperature now shows one decimal instead of two + No issues found after the SQLite package version update ### Inspected following files + SMHIForecastView.js + TemperatureView.js + energySensorMockup.csproj ## What needs to be done + No issues found. ## Conclusion + Works as intended and the code is ready to be merged.

j21marbr commented

2026-05-11 11:23:42 +00:00

Collaborator

No description provided.

Screenshot 2026-05-11 at 13.22.31.png

132 KiB

bild2.png

136 KiB