Naser Mansour
ac22a0b982
Client reported the site's address was missing "Jaffa Street" — added to both Arabic and English footer strings (lang/ar/footer.php, lang/en/footer.php). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
62 lines
2.2 KiB
Markdown
62 lines
2.2 KiB
Markdown
---
|
|
id: 22
|
|
title: 22 Fix Footer Address - Add Missing Street Name
|
|
status: Done
|
|
date: 2026-09-28
|
|
parent: null
|
|
---
|
|
|
|
## Description
|
|
|
|
Client reported the site's footer address is missing the street name. Naser confirmed the correct
|
|
full address (his own words, pasted): "شارع يافا - بجانب الشني / العمارة الأمريكية - الطابق
|
|
الرابع" (Jaffa Street - next to Al-Shini / American Building - 4th Floor). Pure content/copy
|
|
change, no logic.
|
|
|
|
**File to modify:** `lang/ar/footer.php`
|
|
|
|
Current:
|
|
```php
|
|
'address' => 'العمارة الأمريكية - بجانب الشني - الطابق الرابع',
|
|
```
|
|
|
|
Change to:
|
|
```php
|
|
'address' => 'شارع يافا - بجانب الشني - العمارة الأمريكية - الطابق الرابع',
|
|
```
|
|
|
|
**File to modify:** `lang/en/footer.php`
|
|
|
|
Current:
|
|
```php
|
|
'address' => 'American Building - Next to Al-Shini - 4th Floor',
|
|
```
|
|
|
|
Change to:
|
|
```php
|
|
'address' => 'Jaffa Street - Next to Al-Shini - American Building - 4th Floor',
|
|
```
|
|
|
|
No other key in either file changes. This string is rendered in `resources/views/components/footer.blade.php`
|
|
line 24 via `{{ __('footer.address') }}` — no Blade file needs editing, only the two language
|
|
files above.
|
|
|
|
## Acceptance Criteria
|
|
|
|
- [x] `lang/ar/footer.php`'s `address` value equals exactly
|
|
`شارع يافا - بجانب الشني - العمارة الأمريكية - الطابق الرابع`
|
|
- [x] `lang/en/footer.php`'s `address` value equals exactly
|
|
`Jaffa Street - Next to Al-Shini - American Building - 4th Floor`
|
|
- [x] No other key in either file changed (verify: `git diff lang/ar/footer.php lang/en/footer.php`
|
|
shows exactly one changed line per file)
|
|
- [x] Visiting the home page (`/`) still returns HTTP 200 and its rendered HTML contains the new
|
|
Arabic address string (verify with a feature test or an `assertSee` against the existing
|
|
home-page test suite if one covers the footer; if none does, a simple new assertion is fine
|
|
— this is a content check, not new behavior, so TDD's RED/GREEN cycle doesn't apply the
|
|
normal way: writing an assertion that currently fails because the string doesn't match yet
|
|
counts as the "RED" step here)
|
|
|
|
## Sub-tasks
|
|
|
|
None.
|