Skip to content

Remove CA&ES logo from WALTER#265

Open
rmartinsen-ucd wants to merge 1 commit into
mainfrom
rpm/remove-caes-logo
Open

Remove CA&ES logo from WALTER#265
rmartinsen-ucd wants to merge 1 commit into
mainfrom
rpm/remove-caes-logo

Conversation

@rmartinsen-ucd
Copy link
Copy Markdown
Collaborator

@rmartinsen-ucd rmartinsen-ucd commented May 14, 2026

Summary

  • Removed the CA&ES logo from the footer and the about/login page
  • Deleted the unused caes.svg asset

Closes #264

Test plan

  • npm run build passes
  • npm test passes (170/170)
  • Visually confirm footer and /about page render without the logo

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Style
    • Removed CA&ES UC Davis logo and external link from the footer.
    • Removed CA&ES UC Davis logo from the About page and updated layout styling.

Review Change Stack

Per request to promote cross-campus cooperation (issue #264).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 14, 2026

📝 Walkthrough

Walkthrough

The PR removes CA&ES UC Davis logos from two locations: the Footer component's right-side section is replaced with an empty placeholder div, and the About page footer section removes the logo link/image block in favor of a styled paragraph.

Changes

CA&ES Logo Removal

Layer / File(s) Summary
Remove CA&ES logos from footer and about page
web/client/src/components/project/Footer.tsx, web/client/src/routes/about.tsx
Footer right-side content changes from a linked CA&ES logo image to an empty flex-1 div. About page footer section removes the CA&ES logo block and replaces it with a single text-sm text-dark-font/80 styled paragraph.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • ucdavis/Walter#5: Introduced the footer's CA&ES logo block; this PR removes it from the footer and about page.

Poem

🐰 A logo departs, the campus grows wide,
No single emblem now takes the pride,
Cooperation blooms where friction once lay,
WALTER welcomes all, united today! 🌟

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Remove CA&ES logo from WALTER' directly and clearly describes the main change in the pull request.
Linked Issues check ✅ Passed The pull request successfully removes the CA&ES logo from both the footer and about page as required by issue #264.
Out of Scope Changes check ✅ Passed All changes are scoped to removing the CA&ES logo from the footer and about page, directly addressing issue #264 requirements.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch rpm/remove-caes-logo

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

Remove CA&ES logo from WALTER

1 participant