Skip to content

Clean up error allowlist for mock sub_labels route#40

Open
manupawickramasinghe wants to merge 1 commit into
devfrom
fix-sub-labels-allowlist-1900988521162005480
Open

Clean up error allowlist for mock sub_labels route#40
manupawickramasinghe wants to merge 1 commit into
devfrom
fix-sub-labels-allowlist-1900988521162005480

Conversation

@manupawickramasinghe

Copy link
Copy Markdown
Member

Please read the contributing guidelines before submitting a PR.

Proposed change

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code
  • Documentation Update

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to discussion with maintainers (required for any large or "planned" features):

For new features

  • There is an existing feature request or discussion with community interest for this change.
    • Link:

AI disclosure

  • No AI tools were used in this PR.
  • AI tools were used in this PR. Details below:

AI tool(s) used (e.g., Claude, Copilot, ChatGPT, Cursor):

How AI was used (e.g., code generation, code review, debugging, documentation):

Extent of AI involvement (e.g., generated entire implementation, assisted with specific functions, suggested fixes):

Human oversight: Describe what manual review, testing, and validation you performed on the AI-generated portions.

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I can explain every line of code in this PR if asked.
  • UI changes including text have used i18n keys and have been added to the en locale.
  • The code has been formatted using Ruff (ruff format frigate)

Co-authored-by: manupawickramasinghe <73810867+manupawickramasinghe@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 10, 2026 19:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

PR template validation failed

This PR was automatically closed because the description does not follow the pull request template.

Issues found:

  • No Type of change selected. Please check at least one option.
  • No AI disclosure option selected. Please indicate whether AI tools were used.
  • No Checklist items checked. Please review and check the items that apply.

Please update your PR description to include all required sections from the template, then reopen this PR.

If you used an AI tool to generate this PR, please see our contributing guidelines for details.

@github-actions github-actions Bot closed this Jun 10, 2026
@manupawickramasinghe manupawickramasinghe deleted the fix-sub-labels-allowlist-1900988521162005480 branch June 10, 2026 19:28
@ManupaKDU ManupaKDU restored the fix-sub-labels-allowlist-1900988521162005480 branch June 12, 2026 18:50
@ManupaKDU ManupaKDU reopened this Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants