search

eveseat / seat

🌀✳️ SeAT: A Simple, EVE Online API Tool and Corporation Manager
https://eveseat.github.io/docs/
GNU General Public License v2.0
434 stars 142 forks source link

Queue Jobs #38

Closed drkthunder02 closed 8 years ago

drkthunder02 commented 8 years ago

When a user who does not have the permission queue manager, they can see the buttons for queued jobs, working jobs, and error jobs. If the user then pushes the buttons they get an unathorized error message. I think it would be more useful to hide these buttons if the user does not have the correct permission set to view what is contained in the page the buttons link to.

drkthunder02 commented 8 years ago

How is this a wishlist item, when thats how it worked in the 0.x version of SeAT?

Drynael commented 8 years ago

Because this is a completely new SeAT from scratch .

drkthunder02 commented 8 years ago

Yes, but it seems something that should be so simple could be easily fixed. If you send me how do I find the html from the Controllers for these buttons, then I will write the programming myself. But since, I am not good with Laravel, I don't know where these buttons are handled in the vendor packages, therefore, I can't find the buttons to hide them from users who don't have the correct permissions.

warlof commented 8 years ago

I'll take it

leonjza commented 8 years ago

Leave the queue numbers. Just remove the <a> tags if the queue permission is missing.

leonjza commented 8 years ago

Should be fixed in eveseat/web 1.0.19