fix(ui): show permission error when triggering DAG without permission#63018
fix(ui): show permission error when triggering DAG without permission#63018jaymasiwal wants to merge 3 commits intoapache:mainfrom
Conversation
| "intervalStart": "البداية", | ||
| "loading": "جارٍ تحميل معلومات Dag...", | ||
| "loadingFailed": "فشل تحميل معلومات Dag. يرجى المحاولة مرة أخرى.", | ||
| "permissionDenied": "You do not have permission to trigger this DAG.", |
There was a problem hiding this comment.
This one is obviously in wrong place ;-) This is the locale file for Arabic translation.
|
Good catch, thanks! I removed the English message from the Arabic locale file and removed the incorrect English locale file I added earlier. The PR has been updated. |
|
Static checks are failing due to improper indentation for lines 65 through 86. |
|
@jaymasiwal This PR has been converted to draft because it does not yet meet our Pull Request quality criteria. Issues found:
What to do next:
Converting a PR to draft is not a rejection — it is an invitation to bring the PR up to the project's standards so that maintainer review time is spent productively. If you have questions, feel free to ask on the Airflow Slack. |
|
There are conflicts to resolve |
|
This was fixed globally by #61588, closing. Feel free to re-open if I missed something.
|

What this PR does
Improves error handling when triggering a DAG without sufficient permissions.
Previously the UI displayed a generic "Forbidden" error message when the backend returned a 403 response.
This change detects the 403 response and shows a clearer permission-related message in the UI toast.
Changes
useTriggertriggerDag.permissionDeniedmessage instead of generic errorHow it was tested
pnpm build)pnpm lint)