search

apavlidi / IT_API

The Internet services of the IT department of Alexander Technological Education Institute of Thessaloniki
http://api.it.teithe.gr/
MIT License
14 stars 5 forks source link

RSS feed announcements #83

Open kvisnia opened 5 years ago

kvisnia commented 5 years ago

The url link is wrong. Has to be chenged from: https://apps.it.teithe.gr/announcements/ID to https://apps.it.teithe.gr/announcements/announcement/ID

Add attachment tag to the feed.

asidirop commented 5 years ago

Everything has to be changed from apps.it.teithe.gr to apps.iee.ihu.gr :-) You better set the domain as a startup configuration variable

iamaldi commented 5 years ago

The URL you mentioned deviates a little bit outside of REST standards. https://apps.it.teithe.gr/announcements/announcement/ID

Since you want to get an announcement resource by its ID you make a request to the /announcements/ endpoint asking for the resource with ID 12345

https://apps.it.teithe.gr/announcements/12345

I would suggest you keep the current format as it better represents RESTfullnes on your API.