Updated PhD information and order - #18
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the site’s People widget rendering logic and refreshes several author profiles to reflect current group membership, ordering, and profile metadata.
Changes:
- Adjust author ordering/grouping via
weightand move Pavithra to the Alumni group. - Update German’s profile (adds BSc entry and Google Scholar/LinkedIn links).
- Modify the People widget template slicing for middle/alumni/junior rows.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| starters/academic/content/authors/samuel/_index.md | Updates author ordering via weight. |
| starters/academic/content/authors/ritwik/_index.md | Whitespace cleanup in profile front matter. |
| starters/academic/content/authors/pavithra/_index.md | Moves author to Alumni and updates ordering via weight. |
| starters/academic/content/authors/german/_index.md | Updates ordering, adds BSc education entry, and adds social links. |
| starters/academic/content/authors/abhishek/_index.md | Updates author ordering via weight and whitespace cleanup. |
| modules/wowchemy/layouts/partials/widgets/people.html | Updates People widget list slicing for middle/alumni/junior rows. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+70
to
72
| {{ $middleStart := 1 }} | ||
| {{ $middleEnd := sub $authorsCount 4 }} | ||
| {{ if gt $middleEnd $middleStart }} |
| <div class="row row-last" style="margin-top: 15px;justify-content: center;width: -webkit-fill-available;"> | ||
| {{ if eq . "Alumni" }} | ||
| {{ range last 4 $query }} | ||
| {{ range last 5 $query }} |
| {{ else }} | ||
| {{/* Master thesis, Hiwi */}} | ||
| {{ range last 5 $query }} | ||
| {{ range last 3 $query }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary