Skip to content

fix: laravel nova not loading correctly#993

Merged
jxjj merged 3 commits into
developfrom
hotfix/992-nova-not-loading
Jun 8, 2026
Merged

fix: laravel nova not loading correctly#993
jxjj merged 3 commits into
developfrom
hotfix/992-nova-not-loading

Conversation

@jxjj

@jxjj jxjj commented Jun 8, 2026

Copy link
Copy Markdown
Contributor
  • publishes updated Nova files
  • adds a smoke test for Nova (403 if guest, admin gets dashboard and chime page)
  • fixes admin seeder. Incorrect email meant a new admin user was created instead of existing one with global_admin privs.

Resolves #992

On dev.

@jxjj
jxjj requested review from cmcfadden and Copilot June 8, 2026 17:13
@jxjj jxjj self-assigned this Jun 8, 2026

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

Updates the Laravel Nova admin panel assets/config and adds an end-to-end smoke test to prevent regressions where Nova fails to load or authorize correctly.

Changes:

  • Republished Nova frontend assets (mix manifest + compiled JS license updates) and updated Nova translation strings.
  • Updated config/nova.php to match the published Nova configuration (env-driven domain/path and revised middleware configuration).
  • Fixed the admin seed user email to align with Cypress login conventions, and added a Cypress smoke test verifying Nova access control and basic page loading.

Reviewed changes

Copilot reviewed 3 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
resources/lang/vendor/nova/en.json Updates Nova translation strings and fixes/normalizes entries used by newer Nova UI.
public/vendor/nova/vendor.js.LICENSE.txt Reflects updated third-party dependency versions bundled into Nova’s published assets.
public/vendor/nova/ui.js Updated published Nova UI bundle.
public/vendor/nova/mix-manifest.json Updates cache-busting hashes for Nova published assets.
public/vendor/nova/manifest.js Updated published webpack runtime/manifest for Nova assets.
database/seeders/UsersTableSeeder.php Fixes seeded admin email to match the app’s Cypress login/user conventions.
cypress/e2e/ui/nova.test.js Adds Nova smoke tests (guest forbidden, admin can load dashboard + chimes resource).
config/nova.php Aligns Nova config with updated Nova defaults (env variables + middleware changes).

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

@jxjj
jxjj merged commit 180a249 into develop Jun 8, 2026
7 checks passed
@jxjj
jxjj deleted the hotfix/992-nova-not-loading branch June 8, 2026 17:19
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.

Fix admin (Nova)

3 participants