courses: progress reports export button visibility

open-learning-exchange / planet

🌍 Planet Learning - Angular application

https://hub.docker.com/r/treehouses/planet

GNU Affero General Public License v3.0

62 stars 39 forks source link

courses: progress reports export button visibility #7729

Closed Mutugiii closed 2 weeks ago

Mutugiii commented 2 weeks ago

Hide the button when we don't have any exportable data? On click we have a message that there is no valid data. Given that we use the hiding strategy in other parts of the app, which approach should we use?

sahilvunnam commented 2 weeks ago

I think we should hide the button if there is no data to export. Continue previous strategies.

jessewashburn commented 2 weeks ago

I agree with Sahil.

Gavinp14 commented 2 weeks ago

I also agree with Sahil