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

Testing for November 2018 release #160

Closed
4 of 5 tasks
garemoko opened this issue Oct 30, 2018 · 13 comments
Closed
4 of 5 tasks

Testing for November 2018 release #160

garemoko opened this issue Oct 30, 2018 · 13 comments

Comments

@garemoko
Copy link
Collaborator

garemoko commented Oct 30, 2018

@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.

@garemoko garemoko added the 2018 label Oct 30, 2018
@garemoko
Copy link
Collaborator Author

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.

@garemoko
Copy link
Collaborator Author

I pushed a release candidate with the intention of pushing a stable release after testing. https://moodle.org/plugins/pluginversion.php?id=18130

@davidpesce
Copy link
Owner

I won’t be able to until Friday at the earliest. Still at moot and flying back home tomorrow.

@mediabounds
Copy link
Contributor

mediabounds commented Oct 31, 2018 via email

@garemoko
Copy link
Collaborator Author

garemoko commented Nov 1, 2018

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.

@davidpesce
Copy link
Owner

Any chance you can remove the LL integration? Since it's only working with basic for the time being.

@garemoko
Copy link
Collaborator Author

garemoko commented Nov 2, 2018

@davidpesce doesn't it still work with Learning Locker 1?

@garemoko
Copy link
Collaborator Author

garemoko commented Nov 2, 2018

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.

@davidpesce
Copy link
Owner

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?

@davidpesce
Copy link
Owner

I'll check with @ryansmith94 and see about what is needed to get LLv2 working in here.

@caperneoignis
Copy link
Contributor

caperneoignis commented Nov 2, 2018

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.

@garemoko
Copy link
Collaborator Author

garemoko commented Nov 2, 2018

@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.

@garemoko
Copy link
Collaborator Author

garemoko commented Nov 2, 2018

Calling this done, but please continue testing.

@garemoko garemoko closed this as completed Nov 2, 2018
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

4 participants