Skill management

The try block exists to check for the presence of the required fields in
the message.  The ID is converted to ObjectId so that we can quickly tell
if it is even valid to check against the database.


On Thu, Sep 4, 2014 at 6:28 PM, Raymond Chandler III <
notifications@github.com> wrote:

> In plugins/knowledge_tracing.py:
>
> > @@ -27,16 +30,19 @@ def kt_trace(self, message):
> >
> >          try:
> >              sender_entity_id = message["sender_entity_id"]
> > -            skill = message["skill"]
> > +            skill = ObjectId(message["skill"])
>
> Why are we converting this to an ObjectId? Why not just let it be a
> string? Also, why is this named 'skill' and not 'skill_id' if we are only
> accepting IDs?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tutorgen/hpit_services/pull/39/files#r17146824>.
>