-
Notifications
You must be signed in to change notification settings - Fork 0
main/production #17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
main/production #17
Conversation
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 6.2.6 to 6.3.4. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.3.4/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-version: 6.3.4 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
….3.4 Bump vite from 6.2.6 to 6.3.4
Bumps [league/commonmark](https://github.com/thephpleague/commonmark) from 2.6.1 to 2.7.0. - [Release notes](https://github.com/thephpleague/commonmark/releases) - [Changelog](https://github.com/thephpleague/commonmark/blob/2.7/CHANGELOG.md) - [Commits](thephpleague/commonmark@2.6.1...2.7.0) --- updated-dependencies: - dependency-name: league/commonmark dependency-version: 2.7.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…monmark-2.7.0 Bump league/commonmark from 2.6.1 to 2.7.0
Feature About Us
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Adds the About Us section with dynamic contacts, updates navigation to expose the About Us link, and provides new UI components and seeders under the Paperflakes namespace.
- Implements About Us page with controller, ViewDataAction, DTO, seeders, model, migration, and factory
- Updates navigation templates, routes, sitemap, and translations to include About Us
- Introduces reusable components (
list-image-card,list-grid) and contact seeding logic
Reviewed Changes
Copilot reviewed 37 out of 39 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| todo.md | Fix markdown task list layout |
| tests/Pest.php | Update seeder import to new namespace |
| routes/web.php | Un-comment About Us route |
| resources/views/layouts/_partials/_navigation_mobile.blade.php | Un-comment About Us link |
| resources/views/layouts/_partials/_navigation_desktop.blade.php | Re-enable About Us link |
| resources/views/layouts/_partials/_footer.blade.php | Add About Us link to footer |
| resources/views/components/list-image-card.blade.php | Add new card component for contact listings |
| resources/views/components/list-grid.blade.php | Add new grid wrapper component |
| resources/views/components/badge.blade.php | Allow slot content inside badges |
| resources/views/app/services/show.blade.php | Change tag logic for DocuWare |
| resources/views/app/products/show.blade.php | Change tag logic for DocuWare |
| resources/views/app/contact/index.blade.php | Add branch office address section |
| resources/views/app/about-us/index.blade.php | Full rewrite of About Us layout and conditional logic |
| package.json | Bump Vite version |
| lang/en_CH.json | Add “Branch office” translation |
| lang/de_CH.json | Add “Branch office” translation |
| database/seeders/PaperflakesSeeder.php | Add master Paperflakes seeder |
| database/seeders/Paperflakes/ServicesTableSeeder.php | Extend service tags to include DocuWare |
| database/seeders/Paperflakes/RolesAndPermissionsSeeder.php | Namespace adjustment |
| database/seeders/Paperflakes/ProductsTableSeeder.php | Namespace adjustment |
| database/seeders/Paperflakes/PagesTableSeeder.php | Seed About Us page entries |
| database/seeders/Paperflakes/NewsTableSeeder.php | Namespace adjustment |
| database/seeders/Paperflakes/ContactsTableSeeder.php | Add contact records for employees, board, collaborations |
| database/seeders/DatabaseSeeder.php | Use PaperflakesSeeder |
| database/seeders/CodebarSeeder.php | Add placeholder seeder |
| database/migrations/2025_06_23_130054_create_contacts_table.php | Create contacts table |
| database/migrations/2025_04_16_205932_create_product_modules_table.php | Create product_modules table |
| database/factories/ProductModuleFactory.php | Scaffold factory for ProductModule |
| app/Security/Presets/MyCspPreset.php | Allow res.cloudinary.com in CSP |
| app/Models/ProductModule.php | Define ProductModule model and relations |
| app/Models/Product.php | Add productModules relation |
| app/Models/Contact.php | Define Contact model with JSON casts |
| app/Http/Controllers/Sitemap/SitemapController.php | Add about-us.index to sitemap |
| app/Http/Controllers/AboutUs/AboutUsIndexController.php | Inject contacts via ViewDataAction |
| app/Enums/ContactSectionEnum.php | Introduce ContactSectionEnum |
| app/DTO/ContactDTO.php | Map Contact model to DTO |
| app/Actions/ViewDataAction.php | Add contacts method with caching |
Comments suppressed due to low confidence (7)
todo.md:17
- The task list item has an extra hyphen (
--[ ]). It should be formatted as- [ ] Jobs Pageto render correctly.
-[ ] Jobs Page
resources/views/layouts/_partials/_navigation_desktop.blade.php:14
- This line is missing its
<x-a ...component tag opening, leading to a malformed template. Restore the full<x-a :href="localized_route('about-us.index')" label="{{ __('About us') }}"before theclassAttributesattribute.
classAttributes="text-xl md:text-2xl"/>
resources/views/app/about-us/index.blade.php:24
- The condition for the Products section is checking
employee_servicesinstead ofemployee_products. Update the conditional to@if(!empty($contacts->employee_products) && $contacts->employee_products->count()).
@if(!empty($contacts->employee_services) && $contacts->employee_services->count())
resources/views/app/about-us/index.blade.php:49
- The Collaboration section reuses the wrong condition. It should check
!empty($contacts->collaborations)rather thanemployee_services.
@if(!empty($contacts->employee_services) && $contacts->employee_services->count())
app/Actions/ViewDataAction.php:48
- The
CacheandStrfacades are used but not imported. Adduse Illuminate\Support\Facades\Cache;anduse Illuminate\Support\Str;at the top of this file.
return Cache::rememberForever($key, function () use ($locale) {
database/migrations/2025_04_16_205932_create_product_modules_table.php:14
- The
product_modulestable has no foreign key or column forproduct_id, but the model defines abelongsTo(Product::class). Add aunsignedBigInteger('product_id')column (and FK constraint) to match the relation.
Schema::create('product_modules', function (Blueprint $table) {
app/Enums/ContactSectionEnum.php:7
- PHP enums use
casedeclarations, notconst. Changeconst string EMPLOYEE_SERVICES = ...tocase EMPLOYEE_SERVICES = 'employee_services';for a valid string-backed enum.
const string EMPLOYEE_SERVICES = 'employee_services';
No description provided.