As noted in #2831716-50: REST UI does not support "resource" granularity it is unclear what each choice really does. This needs some explanation.
Current text is
Here you can restrict which HTTP methods should this resource support. And within each method, the available serialization formats and authentication providers.
This needs to explain both Granuarity options.
- Resource
- Method
What about
Here you can restrict your resource configuration by either configuring the whole resource or configuring each method separately.
Configuring the whole resource you select which Methods all use the same Accepted request formats using the same Authentication providers.
Configuring by Method you can fine tune which method is using what Accepted request formatseach having their own Authentication providers.
Comment | File | Size | Author |
---|---|---|---|
#7 | add_better_description-2872299-7.patch | 1.79 KB | clemens.tolboom |
#6 | restui-by-resource-by-method.png | 11.95 KB | clemens.tolboom |
#4 | add_better_description-2872299-2.patch | 1.08 KB | clemens.tolboom |
#2 | rest-granuality.png | 50.54 KB | clemens.tolboom |
Comments
Comment #2
clemens.tolboomI don't like the screen space used. Making it one line makes it harder to read. What should we do?
Comment #3
clemens.tolboomComment #4
clemens.tolboomWrong patch
Comment #5
Wim LeersWe should make it more difficult to use
method
granularity, it should become the "advanced" way to configure REST resources.Comment #6
clemens.tolboomNot sure how to make it
?
What about using 'By Resource' and 'By Method'
Comment #7
clemens.tolboomComment #8
Wim Leers#6: we can do that by moving the "by method" configuration to a separate location, hidden behind an "Advanced" tab or something like that.
Comment #9
clemens.tolboomIs the 'advanced' method discouraged then? Otherwise there is no value in hiding it I guess.
Comment #10
Wim LeersYes.
granularity: method
is strongly discouraged, in favor ofresource: granularity
. This has been true since #2721595: Simplify REST configuration.Comment #11
Wim LeersComment #12
clemens.tolboomComment #13
clemens.tolboomIn #3176730: Post update failing for undefined rest_post_update_resource_granularity() upgrading from 8.x to 9.x relates here I guess