Conversation
Reference: #31
Soap2G
added
documentation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR restructures the VRE landing page and supporting documentation by updating team member information, replacing legacy markdown pages with updated MDX content, and adding a new LatestNews component.
- Updated team member descriptions and added a new past member entry
- Replaced an outdated markdown page with a modern MDX landing page and removed the legacy index.md
- Introduced a new LatestNews component with updated news items and refreshed navigation and styling
Reviewed Changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/pages/team.jsx | Updated team member descriptions and added new past member details |
| src/pages/markdown-page.md | Removed obsolete standalone markdown page |
| src/pages/index.mdx | Added new landing page content with updated badges and structured layout |
| src/pages/index.md | Removed old index markdown page |
| src/pages/LatestNews.module.css | Created new CSS module for the LatestNews component |
| src/pages/LatestNews.jsx | Added LatestNews component to display recent news items |
| src/css/custom.css | Updated primary color variables in the custom CSS |
| docusaurus.config.js | Updated nav menu with a new dropdown for VRE components and changed label |
| docs/tech-docs/home.md | Updated documentation header and layout |
| docs/tech-docs/cluster-setup/openstack.md | Added under construction notice |
| docs/tech-docs/cluster-setup/kubernetes.md | Added under construction notice |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR restructures several pages of the Virtual Research Environment (VRE) to update content and improve navigation.
- Introduces new logos, diagrams, and section breakdown on the team and landing pages.
- Removes an unused markdown page and updates navigation links in the configuration.
- Adjusts documentation pages and marks some sections as under construction.
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/pages/team.jsx | Inserts a new team image to be displayed on the team page. |
| src/pages/markdown-page.md | Deletes an obsolete markdown example page. |
| src/pages/index.mdx | Updates layout and content structure of the landing page with new elements and content. |
| src/pages/LatestNews.jsx | Changes news item icons from "file" to "book-open" to better represent the content type. |
| docusaurus.config.js | Adds a dropdown for VRE components and renames the technical documentation label. |
| docs/tech-docs/home.md | Updates the header and introductory text in the technical documentation homepage. |
| docs/tech-docs/cluster-setup/*.md | Adds placeholder pages marked as under construction for OpenStack and Kubernetes setups. |
Comments suppressed due to low confidence (1)
docs/tech-docs/home.md:11
- The word 'infrastrucutre' seems to be misspelled; please update it to 'infrastructure'.
In these pages you will be able to find detailed documentation of how the VRE infrastrucutre is set up.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
garciagenrique
approved these changes
May 13, 2025
Member
garciagenrique
left a comment
garciagenrique
left a comment
There was a problem hiding this comment.
Looks very nice @Soap2G !
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
2 participants
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.
Closes #31
What's new
Major restructuring of the VRE landing page: