wuvt / trackman

WUVT's track logging web-app
GNU Affero General Public License v3.0
4 stars 3 forks source link

"Show all DJs" in login shouldn't show automation #49

Closed echarlie closed 6 years ago

mutantmonkey commented 6 years ago

Please explain why you disagree. It's been this way since I added the "show all DJs" feature; why does it need to change now?

echarlie commented 6 years ago

A user cannot log in as automation, so there's no benefit to displaying it and has the potential to cause confusion.

mutantmonkey commented 6 years ago

Okay. The reason I left it is because it's a way to start automation from the "show all DJs" view, but that makes sense I suppose.

echarlie commented 6 years ago

being able to start automation from the "show all DJs" view wasn't very intuitive to me; I accidentally did it when trying to log in as automation :smiley:

mutantmonkey commented 6 years ago

I fixed this in 77c080f6d052bf597ed512c756138e071874e47a.