Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove or move generic course specific code #9

Open
garemoko opened this issue Jul 21, 2016 · 3 comments
Open

Remove or move generic course specific code #9

garemoko opened this issue Jul 21, 2016 · 3 comments

Comments

@garemoko
Copy link
Contributor

It looks like some code that's specific to a particular ADL course has been included here: 4b1fac2

If this is to be a generic re-usable plugin, those changes either need to be removed from the master branch, or somehow made generic in some way.

@stevenvergenz
Copy link
Contributor

Well, this module really wasn't intended to be more than an internal proof of concept. That whole Converter file was meant to be replaced with every application. You're welcome to adapt this project for your own uses, but ADL has no intention of supporting or developing the bridge any further.

@garemoko
Copy link
Contributor Author

Gotcha. Thanks @stevenvergenz.

Any objections to me continuing to use this issues section and make PRs or would you rather I do that in my own fork?

@stevenvergenz
Copy link
Contributor

I think it would be better if you forked it, to be honest. Since our experiment with the edX platform is complete, you may as well take over the project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants