Skip to content

Comments

Merge newest updates with updated application info#223

Merged
vinisalazar merged 5 commits intomainfrom
dev
Jan 12, 2026
Merged

Merge newest updates with updated application info#223
vinisalazar merged 5 commits intomainfrom
dev

Conversation

@egmg726
Copy link
Contributor

@egmg726 egmg726 commented Jan 12, 2026

I've updated the training program page to include a button to apply for the 2026 cohort

Copy link
Contributor

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 updates the training program page to open applications for the 2026 cohort and makes several improvements to the training materials page. The changes improve navigation usability with semantic IDs, update eligibility requirements, and simplify labeling across the training materials.

Changes:

  • Enabled the application form for the 2026 cohort with a March 6, 2026 closing date
  • Replaced generic tab IDs with semantic names for better maintainability
  • Updated program eligibility requirements to require git and R experience
  • Simplified "R Programming" labels to just "R" throughout training materials
  • Updated and archived the genome browsers tutorial link

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
overrides/training-program.html Opens applications for 2026 cohort, updates tab navigation with semantic IDs, and revises eligibility requirements to require git and R experience
overrides/training-materials.html Simplifies R-related labels, updates genome browsers link to external URL, and moves deprecated link to archive section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

<li class="filtered-listing-item" data-tag="r">
<a href="../../tutorials/seurat-de/seurat-de">
Programming
R
Copy link

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

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

There is inconsistent whitespace formatting. Line 397 has extra spaces before "R" while other similar lines (384, 391) have consistent formatting. This creates visual inconsistency in the rendered list.

Suggested change
R
R

Copilot uses AI. Check for mistakes.
<p>Applications for the 2026 cohort are currently open! Applications close 6th March 2026.</p>
<p>

<a href=https://forms.office.com/r/X5fnjYn6dh>
Copy link

Copilot AI Jan 12, 2026

Choose a reason for hiding this comment

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

The anchor tag is missing quotes around the href attribute value. While this may work in some browsers, it's best practice to properly quote all HTML attribute values. Change href=https://forms.office.com/r/X5fnjYn6dh to href="https://forms.office.com/r/X5fnjYn6dh".

Copilot uses AI. Check for mistakes.
Copy link
Member

@vinisalazar vinisalazar left a comment

Choose a reason for hiding this comment

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

LGTM

@vinisalazar vinisalazar merged commit 51f5492 into main Jan 12, 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