fix: update audit table API permissions so unpublished routes can render PlanningConstraints #2392
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
August flagged this morning that on
/unpublished
routes, PlanningConstraints are failing to render correctly & throwing the following console error:Before: https://editor.planx.uk/barnet/find-out-if-you-need-planning-permission/unpublished
After: https://2392.planx.pizza/barnet/find-out-if-you-need-planning-permission/unpublished
Changes:
planning_constraints_requests
table to grant theapi
role select permissions, consistent with other audit tables, as this is necessary for theinsert_
mutation to return the new record'sid
This bug is only effecting
/unpublished
routes, not/preview
, so no particular rush to deploy