Skip to content

2025-06-28 shift for pp validation and OO cycle commissioning #192

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 11 commits into from
Jul 3, 2025

Conversation

fsoubelet
Copy link
Member

This is a little bit of an ugly entry, since:

  • the shift was (very) long and spanned two different activities
  • there are 2 summaries
  • the summaries are only in OP logbook

I've made a single entry in the table that spans the full length of the shift.
There is a tooltip for each activity separately, and the links refer to the OP logbook summaries.
Tested links in local build, which are fine. To be decided if we want a different way to display this entry.

@fsoubelet fsoubelet self-assigned this Jul 1, 2025
@fsoubelet fsoubelet added Type: Documentation Improvements, updates and fixes to the documentation. Estimate: Normal Straightforward, but might require some time. Probably needs additional tests. Status: Review Needed Work currently stopped, untils someone else reviews it. labels Jul 1, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the LHC 2025 logbook by introducing a new shift entry for Post TS pp Validation and OO Commissioning, adding a results visualization tip, and standardizing link formatting with target="_blank".

  • Add a !!! tip section for results visualization
  • Reformat table rows and apply target="_blank" to external OP links
  • Insert new shift entry on 2025-06-28 with OP-only summary links
  • Extend tooltips with two new activity descriptions
Comments suppressed due to low confidence (1)

@fsoubelet fsoubelet mentioned this pull request Jul 2, 2025
@fsoubelet
Copy link
Member Author

Included MD14963 entry, split last saturday in 2 entries and adopted links from Ewen's PR at #193

Also aextracted the raw links to anchors down below the table as it was getting unreadable.

@fsoubelet fsoubelet requested a review from jgray-19 July 3, 2025 07:30
@fsoubelet fsoubelet merged commit 1da2218 into master Jul 3, 2025
2 checks passed
@fsoubelet fsoubelet deleted the last_shifts branch July 3, 2025 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Estimate: Normal Straightforward, but might require some time. Probably needs additional tests. Status: Review Needed Work currently stopped, untils someone else reviews it. Type: Documentation Improvements, updates and fixes to the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants