Skip to content

chore: add c.com/solutions/education to navigation#16203

Merged
edisile merged 1 commit intomainfrom
WD-34944
Apr 10, 2026
Merged

chore: add c.com/solutions/education to navigation#16203
edisile merged 1 commit intomainfrom
WD-34944

Conversation

@edisile
Copy link
Copy Markdown
Contributor

@edisile edisile commented Apr 2, 2026

do not merge before canonical/canonical.com#2424

meganav copydoc

Done

  • added link to c.com/solutions/education in navigation

QA

  • Open the DEMO
  • Alternatively, check out this feature branch
  • Run the site using the command ./run serve or dotrun
  • View the site locally in your web browser at: http://0.0.0.0:8001/
    • Be sure to test on mobile, tablet and desktop screen sizes
  • check the link to c.com/solutions/education is in the main navigation under Use cases>By industry>Education

Issue / Card

Fixes WD-34933

Screenshots

immagine

Help

QA steps - Commit guidelines

@webteam-app
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.14%. Comparing base (d37f23c) to head (94cf84e).
⚠️ Report is 193 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #16203      +/-   ##
==========================================
+ Coverage   47.75%   48.14%   +0.39%     
==========================================
  Files          37       37              
  Lines        5801     5845      +44     
==========================================
+ Hits         2770     2814      +44     
  Misses       3031     3031              

see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@edisile edisile marked this pull request as ready for review April 7, 2026 07:34
Copilot AI review requested due to automatic review settings April 7, 2026 07:34
Copy link
Copy Markdown
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

Adds an “Education” entry to the main site navigation under Use cases → By industry, pointing to Canonical’s education solutions page.

Changes:

  • Added a new “Education” navigation item under use_case.highlighted_secondary_links -> By industry.
  • Included title, description, and URL for the new entry.

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

Copy link
Copy Markdown
Contributor

@pedoch pedoch left a comment

Choose a reason for hiding this comment

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

LGTM! We should probably confirm the copy as well.

@edisile edisile merged commit c80154d into main Apr 10, 2026
40 of 41 checks passed
@edisile edisile deleted the WD-34944 branch April 10, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants