openHPI / xikolo-android

📱 Android App of the HPI MOOC Platform
https://play.google.com/store/apps/dev?id=7185448023325736337
GNU General Public License v3.0
41 stars 15 forks source link

Disallow enrollments in not enrollable courses #239

Closed bengelhaupt closed 4 years ago

bengelhaupt commented 4 years ago

I placed the the text where the "Enroll" button would normally be and other error messages such as "Not started yet" are displayed.

Fixes #237

mathebox commented 4 years ago

Don't you have an "Enroll" or "Enter course" button also on the course list? These should be hidden as well, right?

Have you tested the behavior for the case that the user has an enrollment to this course? I think the bar will also be shown in this case. Using (!course.enrollable && !course.isEnrolled) as condition might this.