Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RSS feed announcements #83

Open
kvisnia opened this issue Sep 26, 2019 · 2 comments
Open

RSS feed announcements #83

kvisnia opened this issue Sep 26, 2019 · 2 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@kvisnia
Copy link
Collaborator

kvisnia commented Sep 26, 2019

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.

@kvisnia kvisnia added bug Something isn't working enhancement New feature or request labels Sep 26, 2019
@asidirop
Copy link
Contributor

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
Copy link
Contributor

iamaldi commented Oct 14, 2019

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants