WordPress.org

Make WordPress Core

Opened 4 months ago

Last modified 8 weeks ago

#52321 new enhancement

Add site icon to rest index

Reported by: spacedmonkey Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 4.4
Component: REST API Keywords: good-first-bug has-patch has-unit-tests
Focuses: Cc:

Description

Add site icon to list of site metadata return in the index.

Attachments (1)

52321.diff​ (668 bytes) - added by palmiak 2 months ago.

Download all attachments as: .zip

Change History (8)

#1 @TimothyBlynJacobs
4 months ago

  • Keywords good-first-bug added
  • Milestone changed from Awaiting Review to Future Release
  • Version set to 4.4

Should we expose the id of the attachment and then link to it? Or output get_site_icon_url. If the latter, what sizes should we expose?

This ticket was mentioned in ​Slack in #core-restapi by spacedmonkey. ​View the logs.
4 months ago

#3 @palmiak
2 months ago

@spacedmonkey you mean the /wp-json/ url right? Together with name, description etc.

Personally I would use the default size provided by get_site_icon_url - it's filtrable so it wouldn't be a problem if someone would like to change it.

@palmiak
2 months ago

#4 @spacedmonkey
2 months ago

  • Keywords has-patch needs-unit-tests added; needs-patch removed

Thanks for the patch @palmiak.

Looking at the patch, it seems like we should add some basic unit tests. Maybe a case with and without a site icon set to see what happens. The tests should be added ​here.

#5 @lephleg
2 months ago

Hello @palmiak, is there a PR for this in order to add the unit tests?

This ticket was mentioned in ​PR #1111 on ​WordPress/wordpress-develop by ​palmiak.
8 weeks ago

  • Keywords has-unit-tests added; needs-unit-tests removed

https://core.trac.wordpress.org/ticket/52321

#7 @palmiak
8 weeks ago

I added a test to check if key exists and one more, after setting the image, is the image name correct.

Note: See TracTickets for help on using tickets.