main/production #28

StanBarrows · 2025-07-29T23:44:45Z

No description provided.

Copilot

Pull Request Overview

This PR appears to be a significant feature update that separates the news functionality from the homepage and implements various performance optimizations.

Adds dedicated news index page and navigation: Creates a separate news listing page with its own controller and view templates
Introduces new homepage intro component: Implements a configurable welcome section component with company-specific content
Implements performance optimizations: Adds build configurations, resource preloading, and database query improvements

Reviewed Changes

Copilot reviewed 34 out of 37 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
`vite.config.js`	Adds build optimizations including chunk splitting, minification settings, and development server optimizations
`routes/web.php`	Adds news index routes for both English and German locales
`tests/Feature/Controllers/RouteStatusTest.php`	Adds test coverage for the new news.index routes
`resources/views/layouts/app.blade.php`	Adds resource hints and preload directives for performance optimization
`resources/views/layouts/_partials/_navigation*.blade.php`	Updates navigation to include conditional news links and improves mobile layout
`resources/views/app/news/index.blade.php`	Creates dedicated news listing page template
`resources/views/app/start/index.blade.php`	Refactors homepage to include new intro component
`resources/views/components/intro.blade.php`	Implements new configurable intro component with company-specific content
`app/Http/Controllers/News/NewsIndexController.php`	Creates controller for news index functionality
Database seeders and migrations	Updates configuration structure to support new intro component and section toggles
Language files	Adds translations for new UI elements

Comments suppressed due to low confidence (1)

resources/css/app.css:20

The CSS property 'font-preload' does not exist. This appears to be a custom property that may not have any effect. Consider removing it or using a valid CSS property.

    font-preload: true;

Copilot · 2025-07-29T23:45:59Z

vite.config.js

+            },
+        },
+        // Enable source maps for debugging
+        sourcemap: false,


Setting sourcemap to false conflicts with the comment above it that says 'Enable source maps for debugging'. Consider setting this to true or updating the comment to reflect the actual configuration.

Suggested change

sourcemap: false,

sourcemap: true,

Copilot · 2025-07-29T23:46:00Z

resources/views/components/intro.blade.php

@@ -0,0 +1,37 @@
+@use(App\Enums\LocaleEnum;use Illuminate\Support\Facades\Config;use Illuminate\Support\Str;use App\Helpers\HelperMarkdown;use Illuminate\Support\Arr)


The @use directive should have proper semicolon separation between imports. Consider formatting as separate @use statements for better readability.

Suggested change

@use(App\Enums\LocaleEnum;use Illuminate\Support\Facades\Config;use Illuminate\Support\Str;use App\Helpers\HelperMarkdown;use Illuminate\Support\Arr)

@use(App\Enums\LocaleEnum)

@use(Illuminate\Support\Facades\Config)

@use(Illuminate\Support\Str)

@use(App\Helpers\HelperMarkdown)

@use(Illuminate\Support\Arr)

Copilot · 2025-07-29T23:46:00Z

resources/views/components/intro.blade.php

+    };
+    $markdownContent = Arr::get($configuration?->component_intro, $locale);
+    $htmlContent = $markdownContent ? app(HelperMarkdown::class)->formatMarkdown($markdownContent) : '';
+    $htmlContent = preg_replace('/<h2>/', '<h2 class="mb-2 text-lg md:text-xl font-semibold">', $htmlContent);


Using preg_replace to add CSS classes to HTML elements is fragile and error-prone. Consider using a proper HTML parser or moving the styling to CSS classes applied to the container.

Copilot · 2025-07-29T23:46:00Z

resources/views/components/intro.blade.php

+    $markdownContent = Arr::get($configuration?->component_intro, $locale);
+    $htmlContent = $markdownContent ? app(HelperMarkdown::class)->formatMarkdown($markdownContent) : '';
+    $htmlContent = preg_replace('/<h2>/', '<h2 class="mb-2 text-lg md:text-xl font-semibold">', $htmlContent);
+    $htmlContent = preg_replace('/<p>/', '<p class="mb-4">', $htmlContent);


Using preg_replace to add CSS classes to HTML elements is fragile and error-prone. Consider using a proper HTML parser or moving the styling to CSS classes applied to the container.

Copilot · 2025-07-29T23:46:00Z

database/seeders/files/codebar_intro_de.md

+<p>Bei paperflakes unterstützen wir dich dabei, deine Informationsflüsse zu strukturieren und Geschäftsprozesse zu digitalisieren. Wir denken unternehmerisch, beraten mit Weitblick und setzen auf bewährte sowie offene Technologien. Unser Ziel: Lösungen, die auf deine echten Bedürfnisse zugeschnitten sind – und dir im Alltag spürbaren Mehrwert bringen.</p>
+
+<h2>Wie wir arbeiten</h2>
+<p>Am Anfang hören wir dir zu. Denn nur wer Prozesse und Herausforderungen wirklich versteht, kann sinnvolle und nachhaltige Lösungen entwickeln. Gemeinsam mit dir erarbeiten wir ein durchdachtes Konzept, das fachlich und technisch überzeugt. Liegt die Umsetzung in unserem Kompetenzbereich, begleiten wir dich bis zur erfolgreichen Einführung – und darüber hinaus. Falls nicht, sorgen wir dafür, dass deine Anforderungen sauber dokumentiert sind und zuverlässig umgesetzt werden können – von uns oder deinem bevorzugten Partner.</p>


The content refers to 'paperflakes' but this file is named 'codebar_intro_de.md', suggesting it should contain codebar-specific content instead.

Suggested change

<p>Bei paperflakes unterstützen wir dich dabei, deine Informationsflüsse zu strukturieren und Geschäftsprozesse zu digitalisieren. Wir denken unternehmerisch, beraten mit Weitblick und setzen auf bewährte sowie offene Technologien. Unser Ziel: Lösungen, die auf deine echten Bedürfnisse zugeschnitten sind – und dir im Alltag spürbaren Mehrwert bringen.</p>

<h2>Wie wir arbeiten</h2>

<p>Am Anfang hören wir dir zu. Denn nur wer Prozesse und Herausforderungen wirklich versteht, kann sinnvolle und nachhaltige Lösungen entwickeln. Gemeinsam mit dir erarbeiten wir ein durchdachtes Konzept, das fachlich und technisch überzeugt. Liegt die Umsetzung in unserem Kompetenzbereich, begleiten wir dich bis zur erfolgreichen Einführung – und darüber hinaus. Falls nicht, sorgen wir dafür, dass deine Anforderungen sauber dokumentiert sind und zuverlässig umgesetzt werden können – von uns oder deinem bevorzugten Partner.</p>

<p>Bei codebar unterstützen wir dich dabei, Programmierkenntnisse zu erlernen und eine inklusive Tech-Community aufzubauen. Wir fördern Diversität, bieten kostenlose Workshops an und setzen auf eine unterstützende Lernumgebung. Unser Ziel: Menschen zu befähigen, ihre technischen Fähigkeiten zu entwickeln und ihre Karrierechancen zu verbessern.</p>

<h2>Wie wir arbeiten</h2>

<p>Am Anfang steht die Gemeinschaft. Wir hören dir zu, um deine Lernziele und Herausforderungen zu verstehen. Gemeinsam mit dir gestalten wir Workshops und Mentoring-Programme, die auf deine Bedürfnisse zugeschnitten sind. Unsere ehrenamtlichen Mentor:innen begleiten dich auf deinem Weg, und wir schaffen eine Umgebung, in der du dich sicher und unterstützt fühlst.</p>

Copilot · 2025-07-29T23:46:01Z

database/seeders/Paperflakes/PagesTableSeeder.php


                'robots' => 'index,follow',
                'title' => 'Lass uns sprechen',
                'description' => 'Fragen? Wir sind fürr dich da - melde dich jederzeit bei uns.',


There's a typo: 'fürr' should be 'für'.

Suggested change

'description' => 'Fragen? Wir sind fürr dich da - melde dich jederzeit bei uns.',

'description' => 'Fragen? Wir sind für dich da - melde dich jederzeit bei uns.',

StanBarrows added 6 commits July 29, 2025 23:43

wip

81f6286

wip

7796125

wip

4078e3e

wip

bf96ca9

wip

36a1bec

wip

8fde13a

Copilot AI review requested due to automatic review settings July 29, 2025 23:44

Copilot AI reviewed Jul 29, 2025

View reviewed changes

StanBarrows added 3 commits July 30, 2025 01:46

wip

514acaa

wip

826f9cf

wip

70e5797

StanBarrows merged commit bf29a5c into production Jul 29, 2025
5 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

main/production #28

main/production #28

Uh oh!

StanBarrows commented Jul 29, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jul 29, 2025

Uh oh!

Copilot AI Jul 29, 2025

Uh oh!

Copilot AI Jul 29, 2025

Uh oh!

Copilot AI Jul 29, 2025

Uh oh!

Copilot AI Jul 29, 2025

Uh oh!

Copilot AI Jul 29, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -0,0 +1,37 @@
		@use(App\Enums\LocaleEnum;use Illuminate\Support\Facades\Config;use Illuminate\Support\Str;use App\Helpers\HelperMarkdown;use Illuminate\Support\Arr)

-@use(App\Enums\LocaleEnum;use Illuminate\Support\Facades\Config;use Illuminate\Support\Str;use App\Helpers\HelperMarkdown;use Illuminate\Support\Arr)
+@use(App\Enums\LocaleEnum)
+@use(Illuminate\Support\Facades\Config)
+@use(Illuminate\Support\Str)
+@use(App\Helpers\HelperMarkdown)
+@use(Illuminate\Support\Arr)

	'description' => 'Fragen? Wir sind fürr dich da - melde dich jederzeit bei uns.',
	'description' => 'Fragen? Wir sind für dich da - melde dich jederzeit bei uns.',

main/production #28

main/production #28

Uh oh!

Conversation

StanBarrows commented Jul 29, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Jul 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 29, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants