Skip to content

feat(webapp-access): add WEBAPP_PUBLIC_ACCESS_ENABLED toggle#38567

Open
GareArc wants to merge 3 commits into
mainfrom
worktree-esq1-132-webapp-public-access
Open

feat(webapp-access): add WEBAPP_PUBLIC_ACCESS_ENABLED toggle#38567
GareArc wants to merge 3 commits into
mainfrom
worktree-esq1-132-webapp-public-access

Conversation

@GareArc

@GareArc GareArc commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

ESQ1-132. Adds WEBAPP_PUBLIC_ACCESS_ENABLED (default true) to disable the "Public" webapp access-mode option when set false.

GareArc added 2 commits July 8, 2026 04:09
Allows enterprise on-prem deployments to disable the "public" webapp
access-mode choice in the console UI via an opt-out env var (default
true). The flag is env-driven only, with no enterprise-server round
trip, and existing apps already set to public keep displaying that
mode; only future selection is blocked.
The deployments access-control dialog duplicated Infotip's Popover
wiring almost verbatim. overlay.md sanctions Infotip for exactly this
case; the base/* import ban targets modal/dialog/drawer, not Infotip.
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. feat:webapp Ready-to-use AI web app. Also the "Preview" / "Debug & Preview" inside the orchestrate page. labels Jul 8, 2026
@github-actions github-actions Bot added the web This relates to changes on the web. label Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 52.62% 52.62% +0.00%
Strict coverage 52.13% 52.13% +0.00%
Typed symbols 32,506 32,509 +3
Untyped symbols 29,542 29,544 +2
Modules 2977 2978 +1

@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.36%. Comparing base (eca2d41) to head (a940597).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #38567      +/-   ##
==========================================
+ Coverage   85.34%   85.36%   +0.01%     
==========================================
  Files        5013     5012       -1     
  Lines      265942   266059     +117     
  Branches    50558    50563       +5     
==========================================
+ Hits       226980   227128     +148     
+ Misses      34509    34479      -30     
+ Partials     4453     4452       -1     
Flag Coverage Δ
api 85.62% <100.00%> (+0.02%) ⬆️
dify-ui 94.61% <ø> (ø)
web 84.91% <100.00%> (+<0.01%) ⬆️

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

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

🚀 New features to boost your workflow:
  • 📦 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

feat:webapp Ready-to-use AI web app. Also the "Preview" / "Debug & Preview" inside the orchestrate page. size:M This PR changes 30-99 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant