kirstykitto / CLRecipe

A xAPI based Connected Learning Recipe for use with the CLA toolkit
13 stars 3 forks source link

Use activity stream IRIs listed in the registry #4

Open garemoko opened 9 years ago

garemoko commented 9 years ago

I recommend using the activity stream IRIs listed in registry e.g.: https://registry.tincanapi.com/#home/activityTypes

The IRIs you've used do resolve, which is an advantage, but the ones in registry have the bigger advantage of already being potentially in use by adopters as they've been listed on the registry since August 2015.

garemoko commented 9 years ago

I noticed that for verbs this really breaks down as you have the same URL for every verb. I suggest keeping the source column so you can link to the definition of the identifier, but have a new 'identifier' column that uses the IRs from registry.

garemoko commented 9 years ago

Also, in the example it looks like you've used the correct identifiers. i think it's just me getting confused by the lack of 'identifier' column in the tables.

aneesha commented 9 years ago

I've fixed the links in the table.

garemoko commented 9 years ago

Again, this has been done for verbs but not activity types.