Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-29160 META: en language pack improvements
  3. MDL-28176

Rephrase "Enable conditional access" (enableavailability) description

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Language
    • Labels:

      Description

      The description for "Enable conditional access" currently says:
      "When enabled, this lets you set conditions (based on date, grade, or completion) that control whether an activity can be accessed."

      This suggests that conditional access is available for activities but not necessarily for resources. I would suggest to change the text and mention resources as well. I don't think we have a noun that would cover both "activity" and "resource", so the text could simply be:
      "When enabled, this lets you set conditions (based on date, grade, or completion) that control whether an activity/resource can be accessed."

        Gliffy Diagrams

          Activity

          Hide
          salvetore Michael de Raadt added a comment -

          Thanks for reporting this.

          I've put it on our backlog and we'll try to get to it as soon as we can.

          Show
          salvetore Michael de Raadt added a comment - Thanks for reporting this. I've put it on our backlog and we'll try to get to it as soon as we can.
          Hide
          poltawski Dan Poltawski added a comment -

          Hi Helen,

          I was wondering what you thought on this issue?

          Show
          poltawski Dan Poltawski added a comment - Hi Helen, I was wondering what you thought on this issue?
          Hide
          tsala Helen Foster added a comment -

          Hi Dan,

          I agree with Tomasz that language string could be improved by mentioning resources, though I'd suggest the following wording as slightly easier to understand:

          When enabled, this lets you set conditions (based on date, grade, or completion) that control whether an activity or resource can be accessed.

          I don't know why the issue was labeled as 'patch' or set as major priority. As it's an en language pack improvement, I'm setting the component as 'Language' and will move it to MDL-29160 together with other language pack improvements.

          Show
          tsala Helen Foster added a comment - Hi Dan, I agree with Tomasz that language string could be improved by mentioning resources, though I'd suggest the following wording as slightly easier to understand: When enabled, this lets you set conditions (based on date, grade, or completion) that control whether an activity or resource can be accessed. I don't know why the issue was labeled as 'patch' or set as major priority. As it's an en language pack improvement, I'm setting the component as 'Language' and will move it to MDL-29160 together with other language pack improvements.
          Hide
          poltawski Dan Poltawski added a comment -

          Thanks Helen, submittign that for integration

          Show
          poltawski Dan Poltawski added a comment - Thanks Helen, submittign that for integration
          Hide
          poltawski Dan Poltawski 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
          poltawski Dan Poltawski 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
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks (21, 22 & master).

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks (21, 22 & master).
          Hide
          phalacee Jason Fowler added a comment -

          Looks good

          Show
          phalacee Jason Fowler added a comment - Looks good
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          This has been near becoming rejected, because it's not the best code you are able to produce.

          But, luckily, at the end, it has landed and has been spread to all repos out there.

          Many thanks and, don't forget it, keep improving your skills, you can!

          Closing, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - This has been near becoming rejected, because it's not the best code you are able to produce. But, luckily, at the end, it has landed and has been spread to all repos out there. Many thanks and, don't forget it, keep improving your skills, you can! Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/May/12