Skip to content

DDS-014N: Add Dutch Drone Squad about page - #33

Merged
klaasnicolaas merged 4 commits into
mainfrom
codex/dds-014n-about-page
Jul 27, 2026
Merged

DDS-014N: Add Dutch Drone Squad about page#33
klaasnicolaas merged 4 commits into
mainfrom
codex/dds-014n-about-page

Conversation

@klaasnicolaas

@klaasnicolaas klaasnicolaas commented Jul 27, 2026

Copy link
Copy Markdown
Member

Wat is gewijzigd

  • Vervangt de generieke /about-pagina door een eigen Over DDS-pagina.
  • Beschrijft het ontstaan van Dutch Drone Squad, de belangrijkste momenten sinds 2017 en de huidige werkwijze zonder marketingtaal.
  • Voegt de secties voor piloten en bezoekers en het huidige team toe.
  • Maakt de teamweergave op mobiel horizontaal scrollbaar en gebruikt voorlopig portret-placeholders.
  • Behoudt Informatie als navigatiegroep en gebruikt Dutch Drone Squad als footerkop, met Over DDS, Projecten en Partners eronder.
  • Werkt route, SEO en geautomatiseerde controles bij.

Waarom

DDS-014N geeft bezoekers één duidelijke plek om te lezen waar Dutch Drone Squad vandaan komt, hoe de events zijn opgezet en wie ze organiseert. De tekst is bewust feitelijk en compact gehouden.

Impact

De publieke route /about gebruikt voortaan een code-owned Inertia-pagina. De bestaande statische About-configuratie is verwijderd. Andere publieke pagina’s en routes blijven ongewijzigd.

Validatie

  • php artisan test --compact tests/Feature/PublicStaticPagesTest.php — 5 tests, 72 assertions
  • php artisan test --compact tests/Browser/PublicSiteTest.php --filter='about page tells the DDS story' — 1 test, 83 assertions
  • Gerichte mobiele en desktopnavigatietests — 2 tests, 47 assertions
  • npm run types:check
  • npm run build
  • git diff --check

Vervolg

De echte portretfoto’s worden apart opgepakt in #32.

@klaasnicolaas
klaasnicolaas marked this pull request as ready for review July 27, 2026 00:37
Copilot AI review requested due to automatic review settings July 27, 2026 00:37
@klaasnicolaas
klaasnicolaas marked this pull request as draft July 27, 2026 00:41
@klaasnicolaas
klaasnicolaas marked this pull request as ready for review July 27, 2026 00:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Deze PR vervangt de bestaande generieke /about-pagina (config-gedreven public/shell) door een dedicated, code-owned Inertia React pagina (public/about) die het DDS-verhaal, de tijdlijn en het team presenteert, inclusief bijgewerkte navigatie-/footer-groepering en SEO.

Changes:

  • Nieuwe Inertia React pagina resources/js/pages/public/about.tsx met inhoud (hero, oorsprong, tijdlijn, piloten/bezoekers, team) en mobiele teamlijst met horizontale scroll.
  • /about route omgezet naar public/about + SEO aangepast naar titel/omschrijving passend bij de nieuwe pagina.
  • Feature- en browser-tests uitgebreid/aangepast voor nieuwe routecontracten, navigatietekst en inhoudelijke assertions.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/Feature/PublicStaticPagesTest.php Voegt dedicated test toe voor /about als code-owned pagina en verwijdert /about uit de generieke “page contract” dataset.
tests/Browser/PublicSiteTest.php Past navigatie-assertions aan (Informatie → Dutch Drone Squad) en voegt uitgebreide end-to-end test toe voor de nieuwe about page.
routes/web.php Route /about switched van public/shell met config-content naar dedicated public/about met SEO metadata.
resources/js/pages/public/about.tsx Nieuwe about pagina met DDS-verhaal, timeline en teamsectie (incl. placeholders en mobile scroll UX).
resources/js/layouts/public-layout.tsx Wijzigt navigatie-/footer-labeling naar “Dutch Drone Squad” en past header scrolled styling aan.
config/seo.php Update SEO voor about (title/description) naar nieuwe inhoud.
config/public_pages.php Verwijdert legacy about config content.
Comments suppressed due to low confidence (1)

resources/js/pages/public/about.tsx:103

  • gettingStartedIndex({ ... }).url has the same issue as above: it calls the route helper and then reads .url, which is inconsistent with how these helpers are used elsewhere (e.g. gettingStartedIndex.url()), and can produce the wrong value type for PublicHero.

Use gettingStartedIndex.url({ query: ... }) to get a string URL.

                        label: 'Lees hoe je begint',
                        href: gettingStartedIndex({
                            query: { source: 'about-hero' },
                        }).url,
                    },

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

Comment thread resources/js/pages/public/about.tsx
@klaasnicolaas
klaasnicolaas merged commit 058ef71 into main Jul 27, 2026
5 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