Skip to content
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

[Feature Request] Extended Admin Dashboard #146

Closed
ghost opened this issue Sep 3, 2018 · 10 comments
Closed

[Feature Request] Extended Admin Dashboard #146

ghost opened this issue Sep 3, 2018 · 10 comments

Comments

@ghost
Copy link

ghost commented Sep 3, 2018

This will extend the administrative dashboard for more functions.

  • send notifications
  • docs notification system
  • add admins
  • username lookup
  • database access
  • maintenance mode
  • add mod
  • disable user
  • ban ip
  • ban email
  • ban username
  • disable access
  • email verification system
@Pitu
Copy link
Member

Pitu commented Sep 6, 2018

I'm down for:

  • Disabling users
  • Banning ips

The rest is a no-go for now, but you are welcome to submit a PR in the future with that functionality if you feel like coding it. Except the email thing, we don't do emails in lolisafe.

@Pitu Pitu closed this as completed Sep 6, 2018
@Pitu Pitu reopened this Sep 6, 2018
@ghost
Copy link
Author

ghost commented Sep 7, 2018

Alright, thanks for that.

@ghost
Copy link
Author

ghost commented Sep 7, 2018

What about adding additional admins could you do that through the database or what? Instead of having just one root admin user.

@gnosticJade
Copy link

There is no admin column in the users table. You could add it, but then you'd need to add a check for it in the code too. These are the only columns in the users table.
image

@ghost
Copy link
Author

ghost commented Sep 7, 2018

How do you get into the DB?

@ghost
Copy link
Author

ghost commented Sep 8, 2018

@gnosticJade @Pitu I believe more admins should be able to be added. There should be a way and it should be added in the rewrite. This is an enhancement.

@gnosticJade
Copy link

gnosticJade commented Sep 9, 2018

You are more than welcome to add it yourself in a fork and pull request it. There's plenty of other users, such as BobbyWibowo who forked it, added a feature they desired, and pull requested it into this branch.
But realistically, there are slim to no reasons that I can see multiple admins is necessary or high priority.

@ghost
Copy link
Author

ghost commented Sep 9, 2018

@gnosticJade If you have multiple staff members and don't want to login under root. I don't know how to add it that's why I am asking.

@Pitu
Copy link
Member

Pitu commented Sep 12, 2018

For the time being there are no plans to support admins besides the root account. This might change in the future though.

@Pitu Pitu closed this as completed Sep 12, 2018
@jithatsonei
Copy link

only way @Pitu would do this is if you put a PHAT bug bounty on it

@ghost ghost mentioned this issue Sep 21, 2018
2 tasks
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

No branches or pull requests

3 participants