#769-add-energy-view #775

Merged

c24danli merged 2 commits from #769-add-energy-view into team_2_week_5

2026-05-06 11:18:55 +00:00

a24julot commented

2026-05-06 11:06:12 +00:00

Collaborator

Now we can add energyView.

a24julot added 2 commits

2026-05-06 11:06:12 +00:00

Update tooltipConfig.js 088dce9422

Some code was nestled, seperated clock, energy and energyall.

Update tooltipConfig.js c9a5ea861d

Added comment

c24danli commented

2026-05-06 11:12:08 +00:00

Collaborator

Review on #769

Whats done

Tested on Windows 11 using Chrome

App starts
- works well, was a small chage
- Tested the effected views, both energy views and clock which works well

Inspected following files [IMPORTANT!, Don't forget to inspect for unapproved dependencies in package.json ]

frontend/src/configs/tooltipConfig.js
- Indentation/code styling [does it follow prettier]
  - ok
- Naming convention
  - ok
- Comments
  - no need for comments as the code is understandeble
- General code quality
  - ok

Conclusion

Ready for merge

# Review on #769 ## Whats done ### Tested on Windows 11 using Chrome + App starts + works well, was a small chage + Tested the effected views, both energy views and clock which works well ### Inspected following files [IMPORTANT!, Don't forget to inspect for unapproved dependencies in **package.json** ] + frontend/src/configs/tooltipConfig.js + Indentation/code styling [does it follow prettier] + ok + Naming convention + ok + Comments + no need for comments as the code is understandeble + General code quality + ok ## Conclusion + Ready for merge

c24danli approved these changes

2026-05-06 11:12:20 +00:00