search

petersmithca / humhub-api

Implements a JSON RESTful API for HumHub
27 stars 16 forks source link

Add notifications per user #33

Closed petersmithca closed 8 years ago

petersmithca commented 8 years ago

Add a route to get notifications for a specific user.

petersmithca commented 8 years ago

@censtudios - this should add the enhancement you were looking for

Censtudios commented 8 years ago

Looking good. Will update the module on my site later today.

Thanks a lot

Censtudios commented 8 years ago

I uploaded the new files and tried it (/api/notification/752/?access_token=xxx) and I get a "page not found". The notifications for all users worked though.

I will take a look again later when I have more time. Maybe I did something wrong.

petersmithca commented 8 years ago

Maybe try without that last / On May 26, 2016 7:45 AM, "Censtudios" notifications@github.com wrote:

I uploaded the new files and tried it (/api/notification/752/?access_token=xxx) and I get a "page not found". The notifications for all users worked though.

I will take a look again later when I have more time. Maybe I did something wrong.

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub https://github.com/petersmithca/humhub-api/issues/33#issuecomment-221838235

Censtudios commented 8 years ago

I tried with and without

petersmithca commented 8 years ago

hmm odd, was working for me locally. I did have to restart nginx tho for whatever reason. maybe try restarting your webserver? Will have a look again when home as well.

On Thu, May 26, 2016 at 8:23 AM, Censtudios notifications@github.com wrote:

I tried with and without

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub https://github.com/petersmithca/humhub-api/issues/33#issuecomment-221845149

Censtudios commented 8 years ago

I disabled the module and re-enabled it. It works now xD So that was the issue.

And it's indeed without the last /

petersmithca commented 8 years ago

ah ok, it must cache the config I guess! Thanks for the update

On Thu, May 26, 2016 at 8:28 AM, Censtudios notifications@github.com wrote:

I disabled the module and re-enabled it. It works now xD So that was the issue.

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub https://github.com/petersmithca/humhub-api/issues/33#issuecomment-221846060