Skip to content

Notes sent by Eco Ci now shown in frontend #1218

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 6, 2025
Merged

Conversation

ArneTR
Copy link
Member

@ArneTR ArneTR commented Jun 6, 2025

Screenshot 2025-06-06 at 9 32 47 AM

Greptile Summary

Enhances the Eco CI interface by adding support for displaying measurement notes in the frontend, completing the full cycle of note handling from API to UI.

  • Added note field exposure in /api/eco_ci.py GET endpoint to return measurement notes
  • Implemented note display in /frontend/js/ci.js using tooltips for better UX
  • Added 'Note' column to CI run details table in /frontend/ci.html
  • Updated authentication flow in /frontend/request.html to use dedicated authentication page

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

4 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@ArneTR ArneTR merged commit 33e6d8e into main Jun 6, 2025
1 check failed
@ArneTR ArneTR deleted the notes-eco-ci-frontend branch June 6, 2025 07:59
ArneTR added a commit that referenced this pull request Jun 14, 2025
* main: (179 commits)
  Statistical significance is now 10 runs
  Updated Cloud Energy
  Upgraded pandas
  Bump redis from 6.1.0 to 6.2.0 (#1204)
  Bump cachetools from 5.5.2 to 6.0.0 (#1202)
  Bump uvicorn[standard] from 0.34.2 to 0.34.3 (#1210)
  Bump pytest from 8.3.5 to 8.4.0 (#1211)
  Bump python from 3.13.3-slim-bookworm to 3.13.4-slim-bookworm in /docker (#1220)
  Bump aiohttp from 3.11.18 to 3.12.12 (#1221)
  Bump requests from 2.32.3 to 2.32.4 in /docker (#1222)
  Clarified that index.html amounts of data points are user specific
  Keyboard interrupt locally does also not need to lead to error
  Keyboard Interrupt does not need to be alerted in cluster. Is either systemd or manual
  Logout now also inline message; Using dedicated login/logout icons
  Login successful more clear
  Notes sent by Eco Ci now shown in frontend (#1218)
  Sleep time cluster (#1217)
  Rounding with two digits
  Wrong branch for bakerydemo
  Updated CloudEnergy
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant