search

montagejs / screening

49 stars 16 forks source link

Disable the Download All button if there are no scripts in the scripts l... #282

Closed eliseosoto closed 12 years ago

eliseosoto commented 12 years ago

...ist.

Fixes #278

zacharyc commented 12 years ago

I feel like this only takes care of half of the issue, because it does stop the UI. Do you think we should put in some server code that catches an attempt to downloadAll on an empty set, or is the current error sufficient?

eliseosoto commented 12 years ago

I guess you're right. I'm also going to add validation on the server side in case someone wants to use it via the REST API

eliseosoto commented 12 years ago

Ok, I added the validations.