GitLab

[Master] Release: new notifications page

fixes #1521 but no idea why **shrug**

thanks @jwzimmer!

this applies to the comments only

assign invites directly to the state rather than by defaults in order to ensure empty objects can be part of an update

When trying to clear the invite, because invite is currently an object, it’s getting the state’s existing invite re-applied via `defaultsDeep`. Making it an array should solve this issue

this prevents a console error

helps with comment formatting

* Hide messages counter when at 0

fixes #1507

* check for body before looking at success

fixes #1505

Go back to redirecting to splash on logout, and empty messages page too.

this fixes #1500, since lodash works in ie11 but `assign` does not

add comment message to redux to access the current user, and fix the string

this fixes an issue with comment bubble displays in firefox

handles a caching edge case with admin messages.

Pass to legacy on message routes with `ajax`

Currenlty, some legacy endpoints are failing because they’re being directed to www. A similar thing happened with `/explore`, and this is the same fix

Fix GH-1494: fix formatting for dismiss icon, unread message count

fixes #1494

based on https://stackoverflow.com/questions/39009611/react-bindthis-value-in-opposite-order-on-function-declaration

fallout from #1487

* start work on www page
committing out of paranoia.
including changing splash page endpoints

* updates from feedback

thanks @rschamp! This includes:

1. splitting out messages list into a separate component (for clarity)
2. some comment/formatting adjustments for the api calls
3. removal of an extraneous property in emoji-text

* remove duplicate string declaration

* use object.assign instead of defaults deep

we don’t need deep defaults

* fix react warnings

[Develop] Release 2.2.24

[Master] Release 2.2.24

follow up from https://github.com/LLK/scratch-www/pull/1475#issuecomment-326034683