mirror of
https://github.com/discourse/discourse.git
synced 2026-08-09 21:45:25 +08:00
b03ef52553When testing a new LLM configuration in the admin panel, if the test fails (e.g., invalid API key), users see "Target can't be blank" instead of the actual error from the LLM provider. This happens because `track_failures` attempts to create a ProblemCheckTracker using `llm_model.id` as the target, but unsaved models have `id = nil`, which fails the `validates :target, presence: true` validation. The fix adds a `new_record?` check to skip failure tracking for unsaved models, consistent with the existing checks for `blank?` and `seeded?`. Ref - https://meta.discourse.org/t/391497 |
||
|---|---|---|
| .. | ||
| ai_credit_hard_limit.rb | ||
| ai_credit_soft_limit.rb | ||
| ai_llm_status.rb | ||
