411 replace error codes in cards #434

Merged
c24elipe merged 3 commits from 411-replace-error-codes-in-cards into team_2_week_2 2026-04-21 11:02:38 +00:00
Collaborator

Replaced technical error messages with user-friendly messages in cards.

Removed "Fel: {error}" from UI
Added clear, user-friendly messages per feature
No changes to hooks (developers still have access to technical errors)

Updated the error messages in TemperatureView.js and BookingView.js to use user-friendly messages.

Replaced technical error messages with user-friendly messages in cards. Removed "Fel: {error}" from UI Added clear, user-friendly messages per feature No changes to hooks (developers still have access to technical errors) Updated the error messages in TemperatureView.js and BookingView.js to use user-friendly messages.
c24danli left a comment
Collaborator

Review on #411

Whats done

Tested on Windows 11 using Chrome

  • App starts.

  • I have blocked requests from the diffrent URL's to display the error messages

    • Most give the correct error message.
    • TemperatureView.js & BookingView.js have not been changed <p>Fel: {error}</p>;
  • Might be on my end or another issue but i couldnt force the error on the ParkingView it just display the picture without the error.

  • Inspekted files ,WeatherGraph.js, ParkingView.js, ScheduleView.js, SMHIForcastView.js, SMHITemperatureView.js, TemperatureMonthView.js, TemperatureyearView.js,TemperatureTimeSeriesView.js

    • Small chage to all files but looks good
    • Indentation/code styling ok
      • Changed indention on all files from 2 to 4 but no standard in wiki about it so looks good.

What needs to be done

  • TemperatureView.js & BookingView.js
    • [uppdate the error message on these as well

Conclusion

  • fix minor changes before merge
# Review on #411 ## Whats done ### Tested on Windows 11 using Chrome + App starts. + I have blocked requests from the diffrent URL's to display the error messages + Most give the correct error message. + TemperatureView.js & BookingView.js have not been changed `<p>Fel: {error}</p>;` + Might be on my end or another issue but i couldnt force the error on the ParkingView it just display the picture without the error. + Inspekted files ,WeatherGraph.js, ParkingView.js, ScheduleView.js, SMHIForcastView.js, SMHITemperatureView.js, TemperatureMonthView.js, TemperatureyearView.js,TemperatureTimeSeriesView.js + Small chage to all files but looks good + Indentation/code styling ok + Changed indention on all files from 2 to 4 but no standard in wiki about it so looks good. ## What needs to be done + TemperatureView.js & BookingView.js + [uppdate the error message on these as well ## Conclusion + fix minor changes before merge
Collaborator

After seeing that the error messages have been updated, the branch code compiles and runs as intended, so I say it is good to merge.

After seeing that the error messages have been updated, the branch code compiles and runs as intended, so I say it is good to merge.
c24elipe merged commit 02b58e5426 into team_2_week_2 2026-04-21 11:02:38 +00:00
c24elipe deleted branch 411-replace-error-codes-in-cards 2026-04-21 11:02:38 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!434
No description provided.