Skip to content

refactor(admin): eliminate non-filament admin tools - #5200

Open
Jamiras wants to merge 6 commits into
RetroAchievements:masterfrom
Jamiras:refactor/admin_tools
Open

refactor(admin): eliminate non-filament admin tools#5200
Jamiras wants to merge 6 commits into
RetroAchievements:masterfrom
Jamiras:refactor/admin_tools

Conversation

@Jamiras

@Jamiras Jamiras commented Sep 4, 2026

Copy link
Copy Markdown
Member

Moves the last two tools from pages-legacy/admin.blade.php to filament.

@Jamiras Jamiras changed the title refactor(admin) refactor(admin): eliminate non-filament admin tools Sep 4, 2026
@greptile-apps

greptile-apps Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Moves the remaining legacy admin tools into Filament.

  • Adds Copy Unlocks with confirmation.
  • Adds Identify Alts.
  • Removes legacy admin routes and navigation.

Confidence Score: 4/5

Not safe to merge until no-match submissions stop failing.

Copy Unlocks references an undefined variable when no players match. All previous findings are resolved or withdrawn.

Files Needing Attention: resources/views/livewire/administrative-tools/copy-unlocks.blade.php

Reviews (2): Last reviewed commit: "address feedback" | Re-trigger Greptile

Comment thread resources/views/livewire/administrative-tools/identify-alts.blade.php Outdated
Comment thread resources/views/livewire/administrative-tools/identify-alts.blade.php Outdated
Comment thread resources/views/livewire/administrative-tools/copy-unlocks.blade.php Outdated
Comment thread resources/views/livewire/administrative-tools/copy-unlocks.blade.php Outdated
Comment thread resources/views/livewire/administrative-tools/identify-alts.blade.php Outdated
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