cuhacking / CUHackingApp

CUHacking App - 2018 version
2 stars 0 forks source link

[WIP] Add notifications fragment and server side functionality #28

Closed JackMc closed 7 years ago

JackMc commented 7 years ago

What are you trying to accomplish with this PR?

Add support for getting list of notifications from the server. Currently it looks like this:

screen shot 2017-10-28 at 11 35 06 pm

What's done?

How is this accomplished?

Server and client side hackery, unfortunately.

Basically, I added a route /users/:id/notifications which returns all the notifications filtered appropriately for the user with that ID. I then used this route on the client side to construct Notification objects and use them in a ListView.

Checklist

JackMc commented 7 years ago

Merging this for demo purposes, we can work on the crash later.