Skip to content

fix(hub): footer vertical links + all-caps brand name - #118

Merged
MussaJabir merged 1 commit into
developfrom
fix/hub-footer-and-brand-caps
May 27, 2026
Merged

fix(hub): footer vertical links + all-caps brand name#118
MussaJabir merged 1 commit into
developfrom
fix/hub-footer-and-brand-caps

Conversation

@MussaJabir

Copy link
Copy Markdown
Owner

What

Two fixes on the main site (shared core templates):

1. Footer vertical links

The footer "Services" column lists the five verticals — but every entry linked to the generic /services/ hub page. They now honour each Service row's url (already set to /financial/, /construction/, …), so:

Footer label Was Now
Financial Services /services/ /financial/
Construction /services/ /construction/
Energies /services/ /energies/
Farming /services/ /farming/
Investments /services/ /investments/

Falls back to /services/ when a service has no url. (The home page already linked correctly — only the footer was stale.)

2. All-caps brand name

Per the client, the brand name renders in capitals where it's the wordmark — navbar, mobile nav, footer logo, footer copyright — via the upper filter. Page <title>/meta keep their natural case. Vertical legal names are already uppercase, so only the hub is affected.

Checks

  • ruff clean · black --check clean · manage.py check clean · 585 tests pass (added footer link + uppercase tests in apps/hub/tests/test_views.py)

🤖 Generated with Claude Code

Two fixes on the main site, both in the shared core templates:

- Footer "Services" column listed the five verticals but every entry
  linked to the generic /services/ hub page. Honour each Service row's
  `url` (already set to /financial/, /construction/, …) so the links go to
  the right vertical, matching what the home page already does. Falls back
  to /services/ when a service has no url.
- Render the brand name in all capitals where it shows as the wordmark —
  navbar, mobile nav, footer logo, and the footer copyright — via the
  `upper` filter. Title tags and meta keep their natural case. Vertical
  legal names are already uppercase, so only the hub is affected.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@MussaJabir
MussaJabir merged commit 39da34a into develop May 27, 2026
1 check 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.

1 participant