Review contrib-pageLevelProgress README

Description

Review accuracy and quality:
https://github.com/chucklorenz/demo-wiki/wiki/pageLevelProgress-RDME
Feedback goes to Chuck.
Reassign to Chuck.
Chuck push to Adapt github wiki.

Activity

Show:
Tom Greenfield
July 15, 2015, 9:52 AM

Review completed. I've a couple of points:

  1. Is a screenshot being sourced? Only alt text appears at present.

  2. It may be worth mentioning that if the _pageLevelProgress object is omitted from a model then it's the same as _isEnabled being set as false.

Over to Matt for a final check.

Chuck Lorenz
July 15, 2015, 12:37 PM

I chatted with Tom to clarify my understanding of PLP. I have rewritten the Settings Overview. It is substantially different. I am requesting another review, and so have reassigned this to Tom.

Tom Greenfield
July 16, 2015, 10:25 AM

Hi Chuck, I think we still need to reiterate that setting _isEnabled to false at course level disables it everywhere, not just for components.

Chuck Lorenz
July 17, 2015, 12:38 AM

Yes, contentObjects have now been added to the explanation about being disabled.

Chuck Lorenz
July 17, 2015, 12:40 AM

PLP README has been added to develop.

Assignee

Chuck Lorenz

Reporter

Chuck Lorenz

Labels

None

Time tracking

0m

Time remaining

0m

Sprint

None

Priority

Major
Configure