elliotttate / beatsaver-laravel

Rewrite of beatsaver.com (https://github.com/beatsaver/beatsaver) using the laravel framework
https://discord.gg/8B44zqM
MIT License
39 stars 19 forks source link

Feature/admin panel #51

Closed Roderkik closed 5 years ago

Roderkik commented 5 years ago

This PR satisfies the request to have an administration panel for beatsaver (#30)

Functionality

The hide & ban functionality use the soft deletes in laravel to keep things simple.

Changes to existing files/structures:

The UserController got it's login logic abstracted to a separate method (attemptLogin()). It is used in loginSubmit and loginAdminSubmit .

Added barryvdh/laravel-debugbar to the project, for debug purposes. the additions to composer.json can be removed without issue.

Added a migration that adds admin as a field to the users table. This is done to check if the logged in user is an admin and is therefore allowed to access the admin panel.

luludotdev commented 5 years ago

@roderik3000 Songs panel crashes when a song is trying to be shown when the author has been deleted

See here: Crash Details

Roderkik commented 5 years ago

@lolPants Tested & fixed it on my end. can you confirm? 7e9cd4f

luludotdev commented 5 years ago

LGTM 👍

beatsaver commented 5 years ago

One last feature request and Ill import it

All actions taken on the admin panel require a reason (and its stored) and the entire action + reason is posted to the discord webhook (This is to keep transparency)