Skip to content

Fix: preprocessing related fields removed from PATCH endpoint#13112

Merged
dkaushik94 merged 5 commits into
release-1.10.0from
le-1183
May 14, 2026
Merged

Fix: preprocessing related fields removed from PATCH endpoint#13112
dkaushik94 merged 5 commits into
release-1.10.0from
le-1183

Conversation

@dkaushik94
Copy link
Copy Markdown
Member

Removing pre-processing related fields from being updated through the PATCH endpoint since they should be immutable once created.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 13, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 75278b4e-8ae7-4f46-a713-add35d00a7bf

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch le-1183

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

Migration Validation Passed

All migrations follow the Expand-Contract pattern correctly.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.73%. Comparing base (5f1463f) to head (e13d00f).

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##           release-1.10.0   #13112      +/-   ##
==================================================
+ Coverage           54.72%   54.73%   +0.01%     
==================================================
  Files                2145     2145              
  Lines              200205   200199       -6     
  Branches            28489    28490       +1     
==================================================
+ Hits               109562   109586      +24     
+ Misses              89461    89431      -30     
  Partials             1182     1182              
Flag Coverage Δ
backend 59.90% <100.00%> (-0.11%) ⬇️
frontend 54.76% <ø> (+0.04%) ⬆️
lfx 49.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/backend/base/langflow/api/v1/memories.py 86.61% <100.00%> (ø)
...flow/services/database/models/memory_base/model.py 97.75% <ø> (-0.10%) ⬇️
...kend/base/langflow/services/memory_base/service.py 59.85% <100.00%> (+0.83%) ⬆️

... and 48 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label May 13, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 38%
38.4% (47871/124659) 67.9% (6542/9634) 38.28% (1100/2873)

Unit Test Results

Tests Skipped Failures Errors Time
4302 0 💤 0 ❌ 0 🔥 10m 47s ⏱️

@dkaushik94 dkaushik94 enabled auto-merge May 13, 2026 18:06
@dkaushik94 dkaushik94 added this pull request to the merge queue May 13, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 13, 2026
@dkaushik94 dkaushik94 added this pull request to the merge queue May 14, 2026
Merged via the queue into release-1.10.0 with commit e21f588 May 14, 2026
117 of 118 checks passed
@dkaushik94 dkaushik94 deleted the le-1183 branch May 14, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants