Skip to content

fix(tool-admin): remove ToolAnswer records for unlinked catalogs#14

Merged
AdityaKhatri merged 1 commit intodevelopfrom
fix/catalog-question-on-tool-admin
Mar 18, 2026
Merged

fix(tool-admin): remove ToolAnswer records for unlinked catalogs#14
AdityaKhatri merged 1 commit intodevelopfrom
fix/catalog-question-on-tool-admin

Conversation

@sudip-khanal
Copy link
Copy Markdown
Contributor

Changes

  • Add default description when creating ToolAnswer.
  • Remove ToolAnswer entries for questions whose catalogs are no longer linked to the tool

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • n+1 queries
  • flake8 issues
  • print
  • typos
  • unwanted comments

This PR contains valid:

  • tests
  • permission checks (tests here too)

- Add default description when creating ToolAnswer.
- Remove ToolAnswer entries for questions whose catalogs are no longer linked to the tool
@sudip-khanal sudip-khanal force-pushed the fix/catalog-question-on-tool-admin branch from 83bcf16 to 297fe6c Compare March 10, 2026 10:02
@AdityaKhatri AdityaKhatri merged commit 278da35 into develop Mar 18, 2026
2 checks passed
@AdityaKhatri AdityaKhatri deleted the fix/catalog-question-on-tool-admin branch March 18, 2026 05:24
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.

2 participants