-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Route list error and solution #29
Comments
@shone83 Good catch. I added Passport package to support API. If anyone don't want to include this package then just follow these steps
|
And giving which role to edit admin's data would be good? What do you think? And why? Your help will be appreciated. You can add admin edit and translation features and make Pull Request here. |
I've added Passport related instructions in readme.md with a pull request. |
I think that master who create admin can also editing them and delete. Someone must be able to control them. Now you have situation that when admin is created no one can edit anything and what if master mad mistake in the project. Also, admin should have option to edit it's own profile, or just some parts. I know that deleting is a problem because that removes id, but it must be some kind of set admin to non active. That is must have. I planing to translate whole thing to Serbian when all errors are removed. I hope that would be helpfull to motivate others to translate to other languages. |
@shone83 To make any user inactive we need to just set I've started writing test cases. You can write tests too! |
I agree with @shone83 I believe |
@shone83 @norbybaru Now you can activate, deactivate and edit Admin from Master account from view admin page |
This topic can be used until all errors are corrected. The further I noticed is that the count in the dashboard of professors and students does not work. Also in Admin - Profile there is empty filed for nationality and religion. That fields not exist in form when master creating the admin. I do not think it's necessary. |
@shone83 counts in Dashboard are Cached for 24hrs. It should be updated after 24hrs or if you delete cache files. See in |
Yes, I see. These days I will test more and place here if I notice something if that's ok. |
When I run
Also, some columns in users migration is not nullable. It says:
you must add this |
@shone83 in Laravel's document also mentioned that running Dusk is not safe for production environment. To fix this run Can you make a pull request on user migration error? |
As soon as to realize how to do that. I never done it before. |
@shone83 there is a first time for everything 😃 .Try it! |
When I run |
I got another error that |
@ShekhSaifuddin007 Try this solution. If this work let me know. Then I will add this in Readme.md section as instructions. |
@ShekhSaifuddin007 Check this link
|
@changeweb |
@changeweb Btw I sent you an email if you like to check it out. |
php artisan route:list
is not working. Error:Key path "file://C:\xampp\htdocs\musicSchoolDB\storage\oauth-private.key" does not exist or is not readable
You must enter command
php artisan passport:keys
to be able to use route list.@changeweb
Why master can't edit admins? Admin can't edit it's own profile either.
And, do you plane to add translate to other languages?
This is excellent software but I think it need a lot of work. I would love to help.
The text was updated successfully, but these errors were encountered: