Resolve #3472008 "Move jsonapi validation"
1 unresolved thread
Closes #3472008
Merge request reports
Activity
added 1 commit
- 122125ef - chore: Lets check how much time validator takes in tests
added 1 commit
- e555012e - chore: Lets check how much time validator takes in tests by enabling again
added 1 commit
- 24addfc9 - chore: Validate the performance gain with a second run.
added 1 commit
- 34effa8b - chore: Validate the performance gain with a second run. Kube errors made run invalid
added 1 commit
- 83ca0738 - chore: Enable again, validate performance difference
added 1 commit
- fae2b3d0 - feat: add parameter for validation and disable by deffault. Add test to show...
added 1 commit
- 7ee1c77c - fix: add argument to generated expected array
added 1 commit
- 144767ae - fix: add argument to generated expected array
added 1 commit
- b5a30fb5 - fix: when validation is off, expect no error even when invalid response is generated
added 1 commit
- 903a8196 - chore: cleanup test method order and comment
added 46 commits
-
8da7bb91...e1a109a2 - 32 commits from branch
project:11.x
- e1a109a2...a4577f0b - 4 earlier commits
- 0203f5a5 - chore: Enable again, validate performance difference
- d2c1020f - feat: add parameter for validation and disable by deffault. Add test to show...
- 8a347906 - fix: fix argument and codestyle
- d45d7f24 - fix: add argument to generated expected array
- 5599a24b - fix: add argument to generated expected array
- 0804ec6d - fix: different string to unit test
- d5781574 - fix: when validation is off, expect no error even when invalid response is generated
- ba4dd0d6 - fix: remove extra comment
- 7a676052 - chore: cleanup test method order and comment
- 7dd794ac - Update ResourceResponseValidatorTest.php
Toggle commit list-
8da7bb91...e1a109a2 - 32 commits from branch
65 65 tempstore.expire: 604800 66 66 queue.config: 67 67 suspendMaximumWait: 30.0 68 jsonapi.config: 69 validate_schema: false - Comment on lines +68 to +69
Can this go in
jsonapi.services.yml
, we don't want to pollute the core one if we don't have to. Then we shouldn't have to update the scaffold or the default services.yml either.Edited by Dave Long changed this line in version 16 of the diff
added 1 commit
- 8a71e215 - moved flag to jsonapi.services and corrected flagname in files
Please register or sign in to reply