Skip to content

Conversation

@e-kolpakov
Copy link
Contributor

JIRA Ticket: https://openedx.atlassian.net/browse/OSPR-543
Details: See JIRA ticket
Sandbox: LMS, CMS
Merge deadline: Preferably Friday 24th (in time for upcoming release).

Testing instructions:

  1. Set up a couple of mentoring XBlocks
    2, Set up Dashboard block to draw data from those blocks
  2. Set custom values for "Label for average value" for some (or all) mentoring blocks
  3. Publish
  4. Go to LMS, provide answers for mentoring blocks
  5. Navigate to dashboard block

Observe the following:

  • Average values have custom labels specified at step 3.

@e-kolpakov
Copy link
Contributor Author

@sarina follow up PR for configuring individual "Average" labels per mentoring. Updating the sandbox now.

Upd: Sandbox updated

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This reads weirdly in English. Recommend doing, "'exclude_questions' is malformed"

@sarina
Copy link

sarina commented Apr 24, 2015

LGTM. 👍 once wording addressed.

@e-kolpakov e-kolpakov force-pushed the presentation-updates branch 3 times, most recently from 64cbe67 to 5f313e5 Compare April 24, 2015 16:10
@bradenmacdonald
Copy link
Member

@e-kolpakov FYI, the builds are failing due to https://github.com/edx/xblock-sdk/pull/56 . I have a fix at 6a74d6a but it's not on this branch yet.

@bradenmacdonald
Copy link
Member

👍 Once this is rebased so the build passes.

@Kelketek Kelketek force-pushed the presentation-updates branch from 5f313e5 to 5646645 Compare April 24, 2015 16:55
Kelketek added a commit that referenced this pull request Apr 24, 2015
Allowed overriding average labels per mentoring
@Kelketek Kelketek merged commit eb63e06 into edx-release Apr 24, 2015
@Kelketek
Copy link
Member

@antoviaque Merged and ready to go!

Kelketek added a commit to openedx/openedx-platform that referenced this pull request Apr 24, 2015
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

Successfully merging this pull request may close these issues.

5 participants