Moodle
  1. Moodle
  2. MDL-31721

Improve wording for Show the number of correct responses

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.4, 2.2.1, 2.3
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide

      1. Create a matching question.
      2. Check the wording for the 'number of correct responses' under Combined feedback -> partially correct, and under Hint n. Make sure they are reasonably clear.

      Show
      1. Create a matching question. 2. Check the wording for the 'number of correct responses' under Combined feedback -> partially correct, and under Hint n. Make sure they are reasonably clear.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      38309

      Activity

      Hide
      Tim Hunt added a comment -

      Fix ready for integration.

      Show
      Tim Hunt added a comment - Fix ready for integration.
      Hide
      Eloy Lafuente (stronk7) added a comment -

      The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

      TIA and ciao

      Show
      Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
      Hide
      Eloy Lafuente (stronk7) added a comment -

      Just for confirmation, the old string that continues being used @ get_hint_fields() is the desired behavior there, correct?

      Show
      Eloy Lafuente (stronk7) added a comment - Just for confirmation, the old string that continues being used @ get_hint_fields() is the desired behavior there, correct?
      Hide
      Tim Hunt added a comment -

      That is intentional. Using the same string for two different bits of functionality was causing confusion. You will note that the AMOS script does a copy, not a move.

      Show
      Tim Hunt added a comment - That is intentional. Using the same string for two different bits of functionality was causing confusion. You will note that the AMOS script does a copy, not a move.
      Hide
      Eloy Lafuente (stronk7) added a comment -

      Integrated (21, 22 & master), thanks!

      Show
      Eloy Lafuente (stronk7) added a comment - Integrated (21, 22 & master), thanks!
      Hide
      Andrew Davis added a comment -

      Looks good. Passing.

      Show
      Andrew Davis added a comment - Looks good. Passing.
      Hide
      Eloy Lafuente (stronk7) added a comment -

      Well,

      I wish I said it every time
      you do the things you do.
      You always lend a helping hand,
      and I'm filled with gratitude.

      You are strong and generous
      for each and everyone one of us.
      I am eternally grateful,
      I cannot say thanks enough.

      Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

      Show
      Eloy Lafuente (stronk7) added a comment - Well, I wish I said it every time you do the things you do. You always lend a helping hand, and I'm filled with gratitude. You are strong and generous for each and everyone one of us. I am eternally grateful, I cannot say thanks enough. Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

        People

        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: