[FIX] merge conflicts between #823 and #814

misstick commented 8 years ago

[x] when no account found, do not redirect to #mailbox/undefined but account/new,
[x] when refreshing #mailbox/:mailboxID/:messageID : email and list are not visible,
[x] FIX _accounts.get(...) is undefined,

~~- [ ] FIX error_manager.coffee:29 POST http://localhost:9125/activity 413 (Payload Too Large),~~ ~~- [ ] Keyboard navigation is broken~~

misstick commented 8 years ago

@m4dz : is error_manager.coffee:29 POST http://localhost:9125/activity 413 (Payload Too Large) normal? why?

misstick commented 8 years ago

@m4dz can you explain why Keyboard navigation is broken? and why I shouldnt FIX it?

m4dz commented 8 years ago

About unfixed bugs:

"Payload too large" is an error that occurs when React try to POST console.debug logs to the server logger. It never occurs in production mode, as the React console.debug calls are triggered when app run in __DEV__ config, so we can safely ignore them
Keyboard navigation is broken because it is handled via a plugin, and we decided to trash the actuel plugins stack, and 1/ reintroduce it maybe later if needed (with a dedicated API); 2/ add keyboard events supports natively in the app. So feel free to let it unfixed at this time :smiley:

m4dz commented 8 years ago

@misstick can i let you rebase it on development? Thanks!

cozy / cozy-emails