Problem/Motivation

  1. #2721595: Simplify REST configuration introduced 'resource' granularity configuration. It shipped with 8.2.0. 8.3.0 was released last week. Yet REST UI still does not yet support this.
  2. #2831716: REST UI does not support "resource" granularity updates REST UI to support 'resource' granularity.
  3. But what about all the REST resource config that was created in the mean time?

Proposed resolution

#2721595: Simplify REST configuration provided an update path, which did this:

// If each method has the same formats and the same authentication
// providers configured, convert it to 'granularity: resource', which has
// a simpler/less verbose configuration.

IOW: if you have the same formats and the same authentication configured for every method that you have enabled in your 'method' granularity REST resource configuration… then it is converted to 'resource' granularity. It expresses the same, and is not as needlessly complex.

We can simply reuse that same update path in the REST UI module!

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

CommentFileSizeAuthor
#2 2869443-2.patch831 byteswim leers

Comments

Wim Leers created an issue. See original summary.

wim leers’s picture

Status: Active » Needs review
StatusFileSize
new831 bytes
wim leers’s picture

Title: REST UI has been creating 'method' granularity configuration: provide update path to automatically convert to 'resource' granularity where possible » [PP-1] REST UI has been creating 'method' granularity configuration: provide update path to automatically convert to 'resource' granularity where possible
Status: Needs review » Postponed
clemens.tolboom’s picture

Title: [PP-1] REST UI has been creating 'method' granularity configuration: provide update path to automatically convert to 'resource' granularity where possible » REST UI has been creating 'method' granularity configuration: provide update path to automatically convert to 'resource' granularity where possible
Status: Postponed » Needs review
wim leers’s picture

Status: Needs review » Reviewed & tested by the community

And we have a working patch… which just reuses the existing update path in Drupal core, which is thoroughly tested :)

  • Wim Leers authored 7db84d6 on 8.x-1.x
    Issue #2869443 by Wim Leers: REST UI has been creating 'method'...
clemens.tolboom’s picture

Status: Reviewed & tested by the community » Fixed

I haven't tested this patch. It looks OK to me.

wim leers’s picture

Woot! I have tested it, manually. Worked fine :)

I think it makes sense to do a new release now? :)

clemens.tolboom’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.