Sitewide Notices REST API Endpoint is not initialised

[sjregan]

The BP_REST_Sitewide_Notices_Endpoint class is not registered within the messages component rest_api_init() method:

​https://github.com/buddypress/buddypress/blob/master/src/bp-messages/classes/class-bp-messages-component.php#L441

This can be seen by enabling the Messages component and visiting the API endpoint:
/wp-json/buddypress/v1/

The "/wp-json/buddypress/v1/messages" endpoint will be present, but "/wp-json/buddypress/v1/sitewide-notices" will not.

Adds 'BP_REST_Sitewide_Notices_Endpoint' to the list of REST API controllers to initialise for Messages component.
Adds 'BP_REST_Sitewide_Notices_Endpoint' for 'messages' in autoload irregular_map.

Trac ticket: ​https://buddypress.trac.wordpress.org/ticket/8849#ticket

[imath]

Hi @sjregan

Thanks a lot for your feedback and PR. Good catch, fix looks good, I'll give it a last check before committing it asap.

[imath]

Wooo, we even forgot to document the endpoint there: ​https://developer.buddypress.org/bp-rest-api/reference/

😱

[imath]

In 13435:

BP REST API: make sure the SiteWide notice endpoint is available

The corresponding REST Controller wasn't added to list of controllers to
load from the Messages component's rest_api_init() method.

Props sjregan

Closes ​https://github.com/buddypress/buddypress/pull/72
Fixes #8849