Skip to content

add event digital forensics workshop#76

Merged
swillemin-dh merged 6 commits intomainfrom
bit-philology-update
Mar 18, 2026
Merged

add event digital forensics workshop#76
swillemin-dh merged 6 commits intomainfrom
bit-philology-update

Conversation

@swillemin-dh
Copy link
Contributor

@swillemin-dh swillemin-dh commented Mar 17, 2026

Pull request

Proposed changes

Types of changes

  • New feature (non-breaking change which adds functionality).
  • Enhancement (non-breaking change which enhances functionality)
  • Bug Fix (non-breaking change which fixes an issue).
  • Breaking change (fix or feature that would cause existing functionality to change).

Checklist

  • I have read the README document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

Summary by CodeRabbit

  • New Features
    • Added a Digital Forensics Workshop event page with a three-tab interface: Home (overview, registration note, attribution), Program (detailed session schedule with speakers, times, breaks, and PDF program link), and Logistics (travel directions and contact information).

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 17, 2026

Walkthrough

Adds a new Quarto event page for a Digital Forensics Workshop (2026-05-08) with a three-tab layout (Home, Program, Logistics) containing event metadata, a detailed schedule, travel/logistics, and contact/registration notes.

Changes

Cohort / File(s) Summary
Digital Forensics Workshop Event Page
content/events/20260508-digital-forensics-workshop/index.qmd
New Quarto/Markdown event page with YAML front matter (title, date, location, authors), Bootstrap-styled tabbed interface (Home, Program, Logistics). Includes welcome and registration text, embedded HTML/CSS styling, a PDF program link, an image, a detailed schedule table with sessions/speakers/durations/breaks, travel directions, and contact information.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is empty except for template boilerplate; no proposed changes, types of changes, or checklist items are filled in. Fill in the 'Proposed changes' section explaining the workshop details and event structure, select the appropriate change type (likely 'New feature'), and complete relevant checklist items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'add event digital forensics workshop' directly aligns with the changeset, which adds a complete event page for a digital forensics workshop.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bit-philology-update
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

You can enable review details to help with troubleshooting, context usage and more.

Enable the reviews.review_details setting to include review details such as the model used, the time taken for each step and more in the review comments.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 6

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@content/projects/bitphilology/20260508-digital-forensics-workshop/index.qmd`:
- Around line 164-167: The table row with class "highlight-gray" for the 10:15
Coffee Break currently uses colspan="4" which incorrectly spans all columns
including the Time column; update that tr (the row containing <td>10:15</td> and
the centered "Coffee Break (30')" cell) to use colspan="3" so it spans only the
Speaker, Title, and Duration columns.
- Line 74: The aria-labelledby on the tab panel with id "pills-program" is
incorrectly set to "pills-profile-tab"; update the attribute to
aria-labelledby="pills-program-tab" so it matches the actual tab button ID
(change the attribute on the element defined as {`#pills-program` .tab-pane .fade
...}).
- Around line 115-120: Fix the malformed CSS in the .highlight-darkcolor and
.highlight-lightcolor rules: remove the stray "*/" tokens after the semicolons
and correct the invalid hex "#fffff" to a valid 6-digit hex (e.g., "#ffffff");
if you need commented fallback colors, wrap them properly in CSS comments using
/* fallback: `#FAE6A3`; */ or move them to separate properly formatted comments
rather than leaving trailing "*/" fragments so the declarations for
.highlight-darkcolor and .highlight-lightcolor are syntactically valid.
- Around line 178-182: The table row for Kristel Roder is missing its opening
<tr> tag which breaks the table; fix it by inserting a <tr> start tag
immediately before the <td>Kristel Roder</td> cell in the block where the
closing </tr> and the three <td> cells appear (the row containing "Kristel
Roder", "Digitale Archivierung im Schweizerischen Literaturarchiv – aktueller
Stand und Vergehensweisen", and "20'") so the row is properly wrapped with
<tr>...</tr>.
- Around line 1-14: The frontmatter in index.qmd is missing a status field which
breaks the listing sort (content/projects/index.qmd sorts by status asc); add a
status: active entry to the YAML frontmatter of this event (the same value as
the parent Bit Philology project) so it sorts consistently with other research
project entries and mixed event/project listings.
- Line 67: Replace the incorrect internal absolute URL that includes the source
directory ("https://dhbern.github.io/content/projects/bitphilology") with the
correct deployed path ("https://dhbern.github.io/projects/bitphilology/") or,
preferably, a relative link; update the link text in the paragraph that
currently contains that URL so it points to "/projects/bitphilology/" or simply
"projects/bitphilology" (relative) instead of the one containing "/content/".

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2cb88cb2-a45e-4432-9b23-aeb5b723621f

📥 Commits

Reviewing files that changed from the base of the PR and between 3023b03 and 14f696e.

⛔ Files ignored due to path filters (2)
  • content/projects/bitphilology/20260508-digital-forensics-workshop/20260508_DigitalForensicsWorkshop_Program.pdf is excluded by !**/*.pdf
  • content/projects/bitphilology/20260508-digital-forensics-workshop/floppy.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • content/projects/bitphilology/20260508-digital-forensics-workshop/index.qmd

Copy link
Member

@flicksolutions flicksolutions left a comment

Choose a reason for hiding this comment

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

there are some issues to solve before I let you merge

Copy link
Member

Choose a reason for hiding this comment

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

8MB is a bit big a filesize. I don't think you need that high of a resolution for the web ;-)

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@content/events/20260508-digital-forensics-workshop/index.qmd`:
- Around line 116-120: The HTML paragraph block containing the lines with
**Date:**, **Location:** and **Program:** is preventing Quarto/Markdown from
rendering the bold syntax; replace the raw <p> block with Markdown-native
formatting (for example, use plain lines like "**Date:** Friday 8 May 2026"
etc.) or convert the content into proper HTML tags with semantic elements (e.g.,
<strong> Date: </strong>) so the labels render bold; update the section that
currently uses the <p> tag so the Date/Location/Program labels are rendered
correctly.
- Line 60: The referenced image "floppy.png" used in the index.qmd page is
missing; add the actual floppy.png file into the same event directory as that
index.qmd so the markdown image reference ![](floppy.png){fig-align="center"}
can resolve, ensure the file name matches exactly (case-sensitive),
optimize/resize the image as needed for web use, and commit it alongside the
page so the build will render the image.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ba3b3179-d2a2-4944-b4b3-3fc3b2fb9e4e

📥 Commits

Reviewing files that changed from the base of the PR and between 4aa1408 and 432cc43.

⛔ Files ignored due to path filters (2)
  • content/events/20260508-digital-forensics-workshop/20260508_DigitalForensicsWorkshop_Program.pdf is excluded by !**/*.pdf
  • content/events/20260508-digital-forensics-workshop/floppy_150dpi.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • content/events/20260508-digital-forensics-workshop/index.qmd


### 8 May 2026, University of Bern, Switzerland

![](floppy.png){fig-align="center"}
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Verify the referenced image file exists
fd -t f "floppy.png" content/events/20260508-digital-forensics-workshop/

Repository: DHBern/dhbern.github.io

Length of output: 49


Add the missing floppy.png image file to the event directory.

The image referenced at line 60 does not exist at content/events/20260508-digital-forensics-workshop/floppy.png. This will cause the image to fail rendering on the page. Create or provide the file in the event directory.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@content/events/20260508-digital-forensics-workshop/index.qmd` at line 60, The
referenced image "floppy.png" used in the index.qmd page is missing; add the
actual floppy.png file into the same event directory as that index.qmd so the
markdown image reference ![](floppy.png){fig-align="center"} can resolve, ensure
the file name matches exactly (case-sensitive), optimize/resize the image as
needed for web use, and commit it alongside the page so the build will render
the image.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@content/events/20260508-digital-forensics-workshop/index.qmd`:
- Around line 116-120: The <p> HTML block contains a Markdown link "[PDF
version](20260508_DigitalForensicsWorkshop_Program.pdf)" which Quarto won't
render inside raw HTML; update the <p> block in index.qmd so the Program link is
an HTML anchor (e.g., replace the Markdown link with a proper <a href="...">PDF
version</a>) or close the HTML paragraph and move the Markdown link outside it;
target the <p> tag that contains the "Date/Location/Program" lines to make this
change.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 33a508a4-b5fe-469f-86fa-15b772dd7110

📥 Commits

Reviewing files that changed from the base of the PR and between 432cc43 and 207cc15.

⛔ Files ignored due to path filters (1)
  • content/events/20260508-digital-forensics-workshop/floppy_150dpi.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • content/events/20260508-digital-forensics-workshop/index.qmd

Copy link
Member

@flicksolutions flicksolutions left a comment

Choose a reason for hiding this comment

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

approved! great work!

@swillemin-dh swillemin-dh merged commit 74c2d72 into main Mar 18, 2026
7 checks passed
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.

2 participants