diff options
| author | Heiko Bernloehr <Heiko.Bernloehr@FreeIT.de> | 2013-07-18 13:05:45 +0200 |
|---|---|---|
| committer | Heiko Bernloehr <Heiko.Bernloehr@FreeIT.de> | 2013-07-18 23:52:30 +0200 |
| commit | bd3f66df60f551a4349d634cd606990b9f91e87d (patch) | |
| tree | 77219648f602f84af9252ad6197aac692ee94aa0 | |
| parent | 6a2097bcfe56842c861e7f13690c8e2deeb98d8b (diff) | |
| download | ecs2-bd3f66df60f551a4349d634cd606990b9f91e87d.tar.gz ecs2-bd3f66df60f551a4349d634cd606990b9f91e87d.zip | |
Bring back "id" attribute to groups in courses resource.
In my mail to cc-devel mailinglist[1] I wrote:
* We don't need a special "id" attribute for our groups. They are
enumerated implicitely, because they form a sequence. So we could
decide to virtually enumerate them by starting with 0 (zero), i.e. we
could identify the groups with a tuple of
("lectureID",<sequence number of group>), e.g.:
("lsfproxy.uni-stuttgart.de/campusconnect/courses/443889","0")
("lsfproxy.uni-stuttgart.de/campusconnect/courses/443889","1")
("lsfproxy.uni-stuttgart.de/campusconnect/courses/443889","2")
This is because the LMS doesn't give us an unique group id. We get only a
simple enumeration of the available groups. If the count of groups
changes than the enumeration changes.
With hope in the future it is wiser to have such an id.
[1] Date: Sun, 26 May 2013 21:23:30 +0200
From: Heiko Bernloehr <Heiko.Bernloehr@FreeIT.de>
To: campusconnect-devel@freeit.de
Subject: Re: [cc-devel] Simplified course resource
Reply-To: campusconnect-devel@freeit.de
| -rw-r--r-- | campusconnect/schemas/cc_courses.schema.json | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/campusconnect/schemas/cc_courses.schema.json b/campusconnect/schemas/cc_courses.schema.json index 570b043..df40c83 100644 --- a/campusconnect/schemas/cc_courses.schema.json +++ b/campusconnect/schemas/cc_courses.schema.json @@ -92,6 +92,7 @@ "items": { "type":"object", "properties": { + "id": {"type":"string", "required":true}, "title": {"type":"string"}, "comment": {"type":"string"}, "datesAndVenues": { |