-
Notifications
You must be signed in to change notification settings - Fork 0
Add configuration-based links system for conference pages #4
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
Conversation
- Add generic 'links' array structure to conference front matter - Support flexible link naming with automatic grouping by type - Update publication.liquid layout to render links with smart formatting - Group multiple links of same type (e.g., 'Paper on KITopen and IEEE Xplore') - Use 'as...or' for slides/presentations, 'on...and' for papers/packages - Handle conference-specific presentations (e.g., Slides ICSE24, Slides SE25) - Remove hardcoded Links sections from all 8 conference markdown files - Support various link types: Papers, Replication Packages, Slides, Related works - All links verified and working correctly
✅ Deploy Preview for adorable-belekoy-acde02 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Keep the German 'Links und relevante Publikationen' section with its custom formatting for this presentation page.
There was a problem hiding this 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 introduces a configuration-based links system for conference publication pages, replacing hardcoded markdown Links sections with YAML front matter configuration that's automatically rendered by an enhanced publication layout template.
- Adds flexible
linksarray support in conference front matter with automatic grouping and formatting - Migrates 7 conference files (AIRE25, ICSE24, ICSE25, ECSA21, ICSA23, ICSA25, REFSQ25) from hardcoded markdown to YAML configuration
- Implements intelligent link grouping and grammatically correct rendering in
publication.liquidlayout
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
_layouts/publication.liquid |
Adds rendering logic for configuration-based links with automatic grouping by type and smart formatting |
_conferences/refsq25.md |
Migrates hardcoded Links section to YAML front matter configuration |
_conferences/icse25.md |
Migrates hardcoded Links section to YAML front matter configuration |
_conferences/icse24.md |
Migrates hardcoded Links section to YAML front matter configuration including multiple conference-specific slides |
_conferences/icsa25.md |
Migrates hardcoded Links section to YAML front matter configuration |
_conferences/icsa23.md |
Migrates hardcoded Links section to YAML front matter configuration with multiple conference presentations |
_conferences/ecsa21.md |
Migrates hardcoded Links section to YAML front matter configuration |
_conferences/aire25.md |
Migrates hardcoded Links section to YAML front matter configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…e-link case per PR #4 review
dfuchss
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed both review comments in _layouts/publication.liquid:
- Consolidated duplicate slides branches into a single
{% if group_name contains 'Slides' %}block, eliminating duplication and keeping the "as ... or" grammar for all slide groups. - Single-link case was already simplified in the latest branch state. Confirmed it now renders with a single
<li><a ...>{{ link.name }}</a></li>without unnecessary conditionals.
Commit: 492de60
Let me know if you want me to further adjust the grouping wording or add tests/examples in one of the conference pages to demonstrate the variations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
This PR adds a configuration-based links system for conference publication pages, making it easier to maintain and ensuring consistent presentation across all conference pages.
Changes
linksarray to conference front matter supporting multiple link types with custom names_layouts/publication.liquidto automatically render links with smart formattingLink Types Supported
The flexible configuration automatically handles various link types:
Link Type Detection
The system automatically detects link types by parsing the link name:
Examples
AIRE25:
ICSE24:
FG-Arch24: Kept original German "Links und relevante Publikationen" section (unchanged)
Benefits
Verification
✅ 7 conference files migrated and verified
✅ 1 conference file (FG-Arch24) kept in original format
✅ All links migrated correctly
✅ No unintended hardcoded sections removed
✅ All URLs verified and working