BISON-10 - Update Livewire and Activity Log Scaffolding #24
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…vity-logs # Conflicts: # app/Providers/Filament/DashboardPanelProvider.php # composer.json # composer.lock
- Updated package.json to change tailwind version. - Enhanced `theme.css` import flow for larger patterns. - Modify view annotations with BackedEnum for dashboard classes. - switched postcss configuration to respect imported strategy.
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request updates Livewire and Activity Log scaffolding by integrating user activity logging via the spatie/laravel-activitylog package, enhancing the admin dashboard with custom widgets and navigation, and upgrading the frontend build system to Tailwind CSS v4.
Changes:
- Added activity logging for User model tracking changes to name and email fields
- Implemented custom Dashboard page with personalized widgets and quick links
- Upgraded Tailwind CSS from v3 to v4 with new PostCSS configuration
Reviewed changes
Copilot reviewed 28 out of 42 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| composer.json | Added spatie/laravel-activitylog package dependency |
| app/Models/User.php | Integrated LogsActivity trait and configured activity logging options |
| database/migrations/*.php | Created activity_log table and added event/batch_uuid columns |
| app/Filament/Pages/Dashboard.php | Created custom Dashboard page with widgets and sidebar navigation |
| app/Livewire/LinksList.php | Added reusable LinksList widget component |
| resources/views/livewire/links-list.blade.php | Created template for displaying link lists |
| resources/views/filament/pages/dashboard.blade.php | Created custom dashboard layout with sidebar |
| package.json | Upgraded to Tailwind CSS v4 and added related plugins |
| postcss.config.js | Updated PostCSS configuration for Tailwind v4 |
| resources/css/app.css | Simplified CSS imports for Tailwind v4 |
| vite.config.js | Added Filament theme CSS to build inputs |
| public/js/filament/**/*.js | Updated compiled JavaScript assets |
| README.md | Updated documentation URLs for admin and login paths |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| href="https://laravel.com/docs" | ||
| id="docs-card" | ||
| class="flex flex-col items-start gap-6 overflow-hidden rounded-lg bg-white p-6 shadow-[0px_14px_34px_0px_rgba(0,0,0,0.08)] ring-1 ring-white/[0.05] transition duration-300 hover:text-black/70 hover:ring-black/20 focus:outline-none focus-visible:ring-[#FF2D20] md:row-span-3 lg:p-10 lg:pb-10 dark:bg-zinc-900 dark:ring-zinc-800 dark:hover:text-white/70 dark:hover:ring-zinc-700 dark:focus-visible:ring-[#FF2D20]" | ||
| class="flex flex-col items-start gap-6 overflow-hidden rounded-lg bg-white p-6 shadow-[0px_14px_34px_0px_rgba(0,0,0,0.08)] ring-1 ring-white/5 transition duration-300 hover:text-black/70 hover:ring-black/20 focus:outline-none focus-visible:ring-[#FF2D20] md:row-span-3 lg:p-10 lg:pb-10 dark:bg-zinc-900 dark:ring-zinc-800 dark:hover:text-white/70 dark:hover:ring-zinc-700 dark:focus-visible:ring-[#FF2D20]" |
There was a problem hiding this comment.
Changed 'bg-gradient-to-b' to 'bg-linear-to-b', but 'bg-linear-to-b' is not a valid Tailwind CSS class. The correct class is 'bg-gradient-to-b'.
Comment on lines
+74
to
+77
| document.getElementById('screenshot-container').classList.add('hidden!'); | ||
| document.getElementById('docs-card').classList.add('row-span-1!'); | ||
| document.getElementById('docs-card-content').classList.add('flex-row!'); | ||
| document.getElementById('background').classList.add('hidden!'); |
There was a problem hiding this comment.
The class names have incorrect syntax with '!' at the end instead of at the beginning. In Tailwind CSS, the important modifier should be prefixed with '!' (e.g., '!hidden' instead of 'hidden!').
Suggested change
| document.getElementById('screenshot-container').classList.add('hidden!'); | |
| document.getElementById('docs-card').classList.add('row-span-1!'); | |
| document.getElementById('docs-card-content').classList.add('flex-row!'); | |
| document.getElementById('background').classList.add('hidden!'); | |
| document.getElementById('screenshot-container').classList.add('!hidden'); | |
| document.getElementById('docs-card').classList.add('!row-span-1'); | |
| document.getElementById('docs-card-content').classList.add('!flex-row'); | |
| document.getElementById('background').classList.add('!hidden'); |
| </h2> | ||
| @endif | ||
| <ol> | ||
| @foreach ($links as $link ) |
There was a problem hiding this comment.
Extra space before the closing parenthesis in the foreach loop. Should be @foreach ($links as $link) for consistency.
Suggested change
| @foreach ($links as $link ) | |
| @foreach ($links as $link) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces several significant improvements, including the integration of user activity logging, enhancements to the admin dashboard, and updates to frontend dependencies and configuration. The main themes are backend feature expansion (activity logging and dashboard customization) and frontend build system updates.
Backend: User Activity Logging
spatie/laravel-activitylogpackage tocomposer.jsonand implemented activity logging for theUsermodel using theLogsActivitytrait, with configuration to log only changes to thenameandemailfields. [1] [2] [3]activity_logtable, supporting event tracking and batch UUIDs for activity records. [1] [2] [3]Backend: Admin Dashboard Enhancements
Dashboardpage inapp/Filament/Pages/Dashboard.php, featuring personalized headings, widgets (such asAccountWidget), and sidebar quick links for navigation and support resources.LinksListwidget for displaying lists of links in the dashboard sidebar.Frontend: Build System and Styling Updates
@tailwindcss/forms,@tailwindcss/typography,@tailwindcss/postcss,postcss-nesting), updating bothpackage.jsonand the PostCSS configuration to use the new plugin system. [1] [2]Documentation
README.mdto reflect the correct admin and login paths. [1] [2]