-
Notifications
You must be signed in to change notification settings - Fork 39
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
Testing for November 2018 release #160
Comments
Seems to all work for me. @davidpesce @nadavkav @mediabounds @thepurpleblob @caperneoignis Please can you let me know in the next 48 hours if you plan to help out with testing this. If you do, I can wait but if not I'll go ahead and cut a release. |
I pushed a release candidate with the intention of pushing a stable release after testing. https://moodle.org/plugins/pluginversion.php?id=18130 |
I won’t be able to until Friday at the earliest. Still at moot and flying back home tomorrow. |
Sorry, @garemoko — I’m not actively working in Moodle right now so I won’t be able to help test this.
… On Oct 30, 2018, at 9:36 AM, Andrew Downes ***@***.***> wrote:
Seems to all work for me.
@davidpesce @nadavkav @mediabounds @thepurpleblob @caperneoignis Please can you let me know in the next 48 hours if you plan to help out with testing this. If you do, I can wait but if not I'll go ahead and cut a release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Note: we made some changes to the Watershed session API that will break the release candidate. The release candidate should work with watershed production right now. The Github master branch should work with Watershed sandbox. I'm anticipating a couple more changes on the Watershed side and will do a new release of this plugin once that's all finalized on watershed production. |
Any chance you can remove the LL integration? Since it's only working with basic for the time being. |
@davidpesce doesn't it still work with Learning Locker 1? |
The plugin should now be stable in regards to Watershed integration changes (though still waiting on updates to Watershed itself to be pushed from sandbox to production). I'll release a new version once we answer the Learning Locker question. I'm fine with removing it if it's no longer useful. |
Oh, yea, it definitely works on LLv1. I still have a few clients using a customized version of it. Maybe it should specify that it's LLv1? |
I'll check with @ryansmith94 and see about what is needed to get LLv2 working in here. |
I can test it. but I have made changes to the plugin, which I have included in the issues. So I'd have to pull a new instance to test, and only for testing locally but I can make it happen probably next week the earliest though. @garemoko sorry just seeing this now, which github did push notifications to my phone. |
@davidpesce I updated the language file so it should say Learning Locker 1 now. As for what's needed for LL2, see #144 (comment) for some pretty clear instructions. @caperneoignis thanks. I'm going to go ahead with a release of the plugin as we have in master now, but if you notice issues (aside from with completions) I may be able to incorporate them into another release next week. |
Calling this done, but please continue testing. |
@davidpesce @nadavkav @mediabounds @thepurpleblob @caperneoignis
I'm planning to make a new release of this plugin soon and have pushed a number of changes to the master branch. I would ideally like to have the following tested before release:
I plan to test these myself, except for the upgrade tests. Additional testing is appreciated.
Completion tracking is out of scope for this release. I know that there are significant issues with it (#125 #126 #134) and for now at least enabling completion tracking is not supported.
The text was updated successfully, but these errors were encountered: