Skip to content

fix: support Frappe v15 again - #1008

Merged
surajshetty3416 merged 1 commit into
frappe:masterfrom
surajshetty3416:bp/master/frappe-v15-floor
Oct 28, 2026
Merged

surajshetty3416 merged 1 commit into
frappe:masterfrom
surajshetty3416:bp/master/frappe-v15-floor

Conversation

@surajshetty3416

Copy link
Copy Markdown
Member

Backport of #1007 to master. Merging this cuts v1.36.3, the first release since v1.35.2 that installs on Frappe v15 (15.121.3 and later).

version-15 ships the @framework/ui telemetry and TrialBanner the
frontend imports from 15.121.3, so lower the floor to that.

Bench, Pilot and Frappe Cloud parse this range with three different
libraries and no syntax they share can hold a floor per major. v16
benches between 16.0 and 16.36.0 now pass the check and fail the asset
build instead; Pilot reverts such an update on its own.

Also fix the two v15 runtime gaps: frappe.in_test does not exist there
(frappe.flags.in_test works on both), and the editor demo's joined
script query sorted by an ambiguous idx.

Co-Authored-By: Claude <noreply@anthropic.com>
@mergify

mergify Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@surajshetty3416
surajshetty3416 merged commit 46ee62c into frappe:master Oct 28, 2026
6 of 7 checks passed
@surajshetty3416
surajshetty3416 deleted the bp/master/frappe-v15-floor branch October 28, 2026 13:38
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.36.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@codecov

codecov Bot commented Oct 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.54%. Comparing base (daaffa1) to head (0749248).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1008      +/-   ##
==========================================
+ Coverage   75.52%   75.54%   +0.01%     
==========================================
  Files         112      112              
  Lines       12362    12364       +2     
==========================================
+ Hits         9337     9340       +3     
+ Misses       3025     3024       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant