Kyiv
Account created on 11 December 2018, over 6 years ago
#

Recent comments

πŸ‡ΊπŸ‡¦Ukraine ihor_allin Kyiv

Fixed 500 error for the resources without default includes

πŸ‡ΊπŸ‡¦Ukraine ihor_allin Kyiv

Update patch with minor code styles fixes

πŸ‡ΊπŸ‡¦Ukraine ihor_allin Kyiv

For some reason, locally, I have also passed tests by adding a cache tag check: $this->assertSession()->responseHeaderEquals('X-Drupal-Cache', 'HIT') after the second request to '/api/articles'. However, at the same time, when I check it by visiting the JSON API route with the resource with default includes, I always receive an uncached response.

πŸ‡ΊπŸ‡¦Ukraine ihor_allin Kyiv

Fixed bug with variable name

πŸ‡ΊπŸ‡¦Ukraine ihor_allin Kyiv

@bbrala do you have default includes added when you test it locally?

πŸ‡ΊπŸ‡¦Ukraine ihor_allin Kyiv

@bbrala, thank you for the clear explanation. Updating the patch with the corresponding tests. Passed successfully on my local env

πŸ‡ΊπŸ‡¦Ukraine ihor_allin Kyiv

It seems that the responses in these tests do not contain headers, just a raw JSON string.

Production build 0.71.5 2024