search

BYU-ODH / yvideo

4 stars 0 forks source link

Remove "Add to LMS" functionality #10

Closed JKJensen closed 7 years ago

JKJensen commented 8 years ago

Ayamel was originally designed to work with a variety of LMS's, and Ayamel World will continue to support this.

But the BYU instance of Ayamel doesn't need this functionality. Everything we need will come from AIM.

However, in the far flung future, we may want Ayamel to send student data (Time watched, number of words clicked on) directly to Learning Suite as a form of auto-grading.

Should we leave this functionality in, (as in comment it out for now). Or just delete it all?

gliese1337 commented 8 years ago

If you're removing the functionality, then remove the code. You can always get it back later- that's the beauty of version control. Leaving in large chunks of commented out stuff just makes maintenance harder and more confusing.

JKJensen commented 8 years ago

Yes, we will remove LMS functionality, because everything will be routed through AIM from now on.

verdude commented 7 years ago

Made some destructive changes to the database to remove lmskeys from the course table. As a result, all site-permissions are wiped when the commit is applied: 76b490609b91d423dd7fe5654f9a2daa45eb7e3e