[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[RFR] templates://wims-moodle/{templates}



Please find, for review, the debconf templates and packages descriptions for the wims-moodle source package.

This review will last from Wednesday, March 12, 2014 to Saturday, March 22, 2014.

Please send reviews as unified diffs (diff -u) against the original
files. Comments about your proposed changes will be appreciated.

Your review should be sent as an answer to this mail.

When appropriate, I will send intermediate requests for review, with
"[RFRn]" (n>=2) as a subject tag.

When we will reach a consensus, I send a "Last Chance For
Comments" mail with "[LCFC]" as a subject tag.

Finally, a summary will be sent to the review bug report,
and a mail will be sent to this list with "[BTS]" as a subject tag.

Rationale:
--- wims-moodle.old/debian/templates	2014-03-08 18:00:45.151242468 +0100
+++ wims-moodle/debian/templates	2014-03-12 19:09:29.285610765 +0100
@@ -1,12 +1,13 @@
 Template: wims-moodle/remoteurl
 Type: string
 Default: https://wims.domain
-_Description: Please type the base address to access Wims from remote machines:
- The base address to access Wims will be needed to enable users to access
- their Wims assignments from Moodle.
+_Description: URL to access Wims from remote machines:
+ Please enter the URL that should be used to access Wims, so that
+ users can get their Wims assignments from Moodle.
  .
- If you can access Wims at https://wims.example.org/wims/, you should
- answer https://wims.example.org
+ Do not specify the path, but only the protocol and domain name. For
+ instance, if Wims is accessed from https://wims.example.org/wims/, then
+ you should enter https://wims.example.org only.

The question here is not really clear, which makes the template
writing a bit cumbersome. I think I understand the underlying idea,
but I suspect users might be confused. Still, I tried to reformulate
according to our usual writing style.


  .
  Please notice that the protocol should be https. Apache must be configured
- to be able to answer https requests.
+ to be able to answer HTTPS requests.

Just turning the protocol part in capitals as this is an acronym.

I didn't really reviewed the package description part in
debian/control. It seems fine by me, though. Justin might maybe suggest
a few changes.

-- 


Template: wims-moodle/remoteurl
Type: string
Default: https://wims.domain
_Description: URL to access Wims from remote machines:
 Please enter the URL that should be used to access Wims, so that
 users can get their Wims assignments from Moodle.
 .
 Do not specify the path, but only the protocol and domain name. For
 instance, if Wims is accessed from https://wims.example.org/wims/, then
 you should enter https://wims.example.org only.
 .
 Please notice that the protocol should be https. Apache must be configured
 to be able to answer HTTPS requests.
--- wims-moodle.old/debian/templates	2014-03-08 18:00:45.151242468 +0100
+++ wims-moodle/debian/templates	2014-03-12 19:09:29.285610765 +0100
@@ -1,12 +1,13 @@
 Template: wims-moodle/remoteurl
 Type: string
 Default: https://wims.domain
-_Description: Please type the base address to access Wims from remote machines:
- The base address to access Wims will be needed to enable users to access
- their Wims assignments from Moodle.
+_Description: URL to access Wims from remote machines:
+ Please enter the URL that should be used to access Wims, so that
+ users can get their Wims assignments from Moodle.
  .
- If you can access Wims at https://wims.example.org/wims/, you should
- answer https://wims.example.org
+ Do not specify the path, but only the protocol and domain name. For
+ instance, if Wims is accessed from https://wims.example.org/wims/, then
+ you should enter https://wims.example.org only.
  .
  Please notice that the protocol should be https. Apache must be configured
- to be able to answer https requests.
+ to be able to answer HTTPS requests.
Source: wims-moodle
Section: web
Priority: extra
Maintainer: Georges Khaznadar <georgesk@debian.org>
Build-Depends: debhelper (>= 9), po-debconf
Standards-Version: 3.9.5
Homepage: http://wiki.ofset.org/index.php/Access_a_Wims_server_through_a_CMS

Package: wims-moodle
Architecture: all
Depends: ${misc:Depends}, wims (>= 4.00)|wims-installed-manually, pwgen
Suggests: moodle (>= 2.2.1)|moodle-installed-manually
Description: module for Moodle featuring Wims mini-classes
 Moodle (Modular Object-Oriented Dynamic Learning Environment) is a course
 management system - a software package designed to help educators create
 quality online courses.
 .
 Wims is an educational platform originally developed for mathematics
 but now also supporting subjects such as physics, chemistry, biology,
 and languages. Its more striking feature is to provide a huge quantity
 of interactive exercises with randomly assigned data, allowing the
 learners to work on the same type of exercise but with different
 data and as often as they want.
 .
 This module adds the possibility to define Wims classes as special
 assignments in Moodle. Wims classes come in two flavors: one is
 meant to contain a set of worksheets (interactive randomly generated
 exercices for the students), and the other one is meant to contain
 exams. The content of exams are based on previously published
 worksheets.
 .
 The students already logged in Moodle are automatically logged into Wims,
 and as they close the Wims session they entered, their scores in Moodle are
 updated.

Attachment: signature.asc
Description: Digital signature


Reply to: