Merge pull request #6937 from pixelfed/shleeable-patch-1

Update README.md
pull/6938/head^2
Shlee 4 weeks ago committed by GitHub
commit 0497aab3d2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

@ -137,51 +137,3 @@ Read-only inspectors for troubleshooting. They do not modify data.
| `user:2fa` | Disable two-factor authentication for a username. |
| `user:unsuspend` | Unsuspend a local user. |
| `user:verifyemail` | Verify a user's email address. |
---
## Audit notes
No commands were found to be **broken**: every `handle()` body is functional,
all referenced classes/config resolve (`App\Util\Media\Filter`, the media
`filter_class` column, the `s3-old` disk, and `config('import.instagram')` all
exist), and there are no duplicate command names.
The items below are cleanup opportunities, not failures. Nothing here is deleted
automatically — these are recommendations for a maintainer to confirm.
### Likely obsolete (legacy one-off migrations)
These were written to repair specific historical data states and are unlikely to
be needed on a current install. Candidates for removal after confirming they are
no longer required:
- `FixBugs/MediaFix.php` (`media:fix`) — repairs image-filter data and refuses to
run below v0.10.8. Image filters are a deprecated feature.
- `FixBugs/StatusDedupe.php` (`status:dedup`) — dedupes statuses created *before*
the unique-URI migration; not compatible with Postgres.
- `FixBugs/AvatarDefaultMigration.php` (`fix:avatars`) — replaces old SVG
identicon avatars, a long-since-removed avatar style.
### Missing descriptions (metadata only)
These work but still carry the scaffold default `"Command description"`, so they
read poorly in `php artisan list`. Worth filling in:
- `Admin/CaptchaToggleCommand.php` (`app:captcha-toggle-command`)
- `FixBugs/FetchMissingMediaMimeType.php` (`app:fetch-missing-media-mime-type`)
- `FixBugs/FixMissingUserProfile.php` (`app:fix-missing-user-profile`)
- `FixBugs/HashtagRelatedGenerate.php` (`app:hashtag-related-generate`)
- `Internal/CleanupExpiredAppRegistrations.php` (`app:cleanup-expired-app-registrations`)
- `Internal/ImportRemoveDeletedAccounts.php` (`app:import-remove-deleted-accounts`)
- `Internal/ImportUploadCleanStorage.php` (`app:import-upload-clean-storage`)
- `Internal/ImportUploadGarbageCollection.php` (`app:import-upload-garbage-collection`)
### Naming inconsistencies
- The three media-move commands use PascalCase signatures
(`admin:MediaMoveStorageCloudToCloud`, `...CloudToLocal`, `...LocalToCloud`),
which breaks the kebab-case convention used elsewhere. Consider
`admin:media-move-storage-*`.
- Many commands use the generic scaffold prefix `app:` rather than a meaningful
namespace (`user:`, `media:`, `gc:`, etc.). Not harmful, but inconsistent.

Loading…
Cancel
Save