#39178 closed enhancement (fixed)
REST API: Improve strings added after 4.7 string freeze
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.8 | Priority: | normal |
Severity: | normal | Version: | 4.7 |
Component: | REST API | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
We fixed a few API bugs in 4.7 that required new error strings. Some of these were during string freeze, so we repurposed existing strings rather than adding new ones.
We should go through and update these to better messages and error codes. For example, [39440] added rest_orderby_include_missing_include
which needs a more helpful message and a less verbose error code (see the rest_no_search_term_defined
error immediately above).
Attachments (2)
Change History (15)
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
8 years ago
#3
@
8 years ago
- Milestone changed from 4.7.1 to 4.8
Given that this is more of a "nice to have" and not a breaking issue, we're going to plan on resolving this in 4.8. See linked Slack discussion for conversation during/after today's bug scrub.
#5
@
8 years ago
- Keywords needs-testing added
This needs testing and an owner if its going to land in 4.8. Anyone care to help review this to see it land in 4.8?
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
8 years ago
#8
@
8 years ago
I will test and commit what we have so far, are there any other strings we need to adjust @jnylen0?
#9
@
8 years ago
39178.2.diff - slight language fix define a include
changed to define an include
Note that we usually don't change strings in minor releases.