Activisme-be / Activisme-be/Spoon

Cruddy by design

Offen
#144 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
PHP
Sterne
0
Forks
2
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

:warning: The following controllers contain actions outside of the 7 resource actions (`index`, `create`, `store`, `show`, `edit`, `update`, `destroy`). For more details, review [the docs](https://laravel.com/docs/5.7/controllers#resource-controllers) or watch [Cruddy by Design](https://www.youtube.com/watch?v=MF0jFKvS4SI) to see if you can rework these into *resource controllers*.

- [ ] app/Http/Controllers/ActivityController.php
- [ ] app/Http/Controllers/Auth/PasswordSecurityController.php
- [ ] app/Http/Controllers/Auth/TwoFactorResetController.php
- [ ] app/Http/Controllers/HomeController.php
- [ ] app/Http/Controllers/NotificationController.php
- [ ] app/Http/Controllers/Users/AccountController.php
- [ ] app/Http/Controllers/Users/IndexController.php

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Open each listed file under app/Http/Controllers (ActivityController, Auth/PasswordSecurityController, Auth/TwoFactorResetController, HomeController, NotificationController, Users/AccountController, Users/IndexController) and list the public actions. Compare those methods with Laravel’s 7 resource actions from the linked docs and refactor controllers to match that shape where appropriate. No specific test is named; run the project tests (or any controller/route checks) after changes and confirm each listed controller follows resource-controller behavior.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
laravel, php
Bereich
backend
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
52/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.