- Issue created by @danielveza
- Assigned to Akhil Babu
- Merge request !260#3472193: Add hook_requirements() warning for sites with JSON:API, error if assertions are enabled → (Merged) created by Akhil Babu
- Issue was unassigned.
- Status changed to Needs review
6 months ago 9:37am 5 September 2024 - Status changed to Needs work
6 months ago 10:50am 5 September 2024 - Status changed to Needs review
6 months ago 12:41pm 5 September 2024 - 🇮🇳India Akhil Babu Chengannur
Thanks @wim leers, I've used the same logic in
\Drupal\experience_builder\EventSubscriber\ApiMessageValidator::isProd()
to check if assertions are enabled.- Should we change this to a common function? (
ApiMessageValidator::isProd()
) is a private method - Should the priority be changed to
REQUIREMENT_ERROR
if assertions are enabed?
- Should we change this to a common function? (
- Status changed to Needs work
6 months ago 1:48pm 5 September 2024 - Assigned to Akhil Babu
- Issue was unassigned.
- Status changed to Needs review
6 months ago 6:14am 6 September 2024 - Status changed to RTBC
6 months ago 12:39pm 6 September 2024 - Status changed to Fixed
6 months ago 12:46pm 6 September 2024 -
wim leers →
committed ddc20bd9 on 0.x authored by
akhil babu →
Issue #3472193 by akhil babu, wim leers, danielveza, larowlan: Add...
-
wim leers →
committed ddc20bd9 on 0.x authored by
akhil babu →
Automatically closed - issue fixed for 2 weeks with no activity.