#769-add-energy-view #773

Merged

a24julot merged 2 commits from #769-add-energy-view into team_1_week_5

2026-05-06 11:15:19 +00:00

a24julot commented

2026-05-06 10:56:30 +00:00

Collaborator

Now energyView can be added without errors.

a24julot added 2 commits

2026-05-06 10:56:30 +00:00

Update tooltipConfig.js 088dce9422

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

Update tooltipConfig.js c9a5ea861d

Added comment

a24julot

2026-05-06 10:57:03 +00:00

added the
Under-review
label
requested review from c24danli

c24danli commented

2026-05-06 11:11:41 +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:45 +00:00

a24julot merged commit b7acfec68e into team_1_week_5

2026-05-06 11:15:19 +00:00

a24julot referenced this pull request from a commit

2026-05-06 11:15:19 +00:00

Merge pull request '#769-add-energy-view' (#773) from #769-add-energy-view into team_1_week_5