oSoc14 / code9000

Calendar app for schools
MIT License
1 stars 4 forks source link

iCal route params #11

Closed bjorvack closed 10 years ago

bjorvack commented 10 years ago

The url for the .ics file should be yoururl.com/school/group/cal.ics. however the school and/or group names aren't unique yet.

Dobbelsteen commented 10 years ago

We should be able to find a way to identify each unique school , even if they have similar names. For example by a unique code that will be generated, but something readable would be best.

coreation commented 10 years ago

A name + _{id} could do the trick, just make sure when you choose a strategy, you make it somewhere abstract. Meaning put it in a function somewhere, and don't hard code it 5 times in different controllers for example, that way you can make a preliminary decision and come back to it when better solutions arise.

On Fri, Jul 4, 2014 at 2:19 PM, Dobbelsteen notifications@github.com wrote:

We should be able to find a way to identify each unique school , even if they have similar names. For example by a unique code that will be generated, but something readable would be best.

— Reply to this email directly or view it on GitHub https://github.com/oSoc14/code9000/issues/11#issuecomment-48037446.

Met vriendelijke groeten,

Jan Vansteenlandt

Dobbelsteen commented 10 years ago

Good idea, that's porbably the best/most readable solution. We'll try to implement it in due time

coreation commented 10 years ago

Since you already have a URI strategy that's being used now, can this be closed @Dobbelsteen ?

Dobbelsteen commented 10 years ago

Yea, this is indeed finished. I wanted to keep it open as a reminder for the other export possibilities, but I suppose that's a feature by itself :)