search

reddit / node-api-client

MIT License
90 stars 30 forks source link

Model fixes #173

Closed uzi closed 3 years ago

uzi commented 8 years ago

This fixes a bit of data that api-client sends back. Note that there should be a few more fixes for the mail stuff before this goes in:

1) We'll want to POST to /api/read_message ids of messages as we read them. This can be done in reddit-mobile, but it might be better to do here in node-api-client in MessagesEndpoint.es6.js ... probably in the get() method when there's a specific thread we're fetching. 2) We'll want to update reddit-mobile's messaging views to highlight (or embolden) messages that are new. 3) We'll need some badge css styling for reddit-mobile in the SettingsOverlayMenu where we show inboxCount.

I consider this a WIP until those things are done as well.

phil303 commented 8 years ago

@uzi besides the typo, is this just a prep patch?

uzi commented 8 years ago

Yeah... just wanted to get these things captured in case someone else has a chance to take a stab at any of the three things I mentioned... or at leas to have the discussion with folks.