feat: License change on AppBuilder by kavinvenkatachalam · Pull Request #14391 · ToolJet/ToolJet

kavinvenkatachalam · 2025-10-23T05:29:32Z

This pull request refactors how license and feature access information is accessed throughout the frontend codebase. The main change is to consistently use the new useLicenseStore hook to retrieve featureAccess and related license status, instead of accessing it directly from the main application store (useStore). This improves code modularity and centralizes license management. Additionally, some unused imports and variables are cleaned up.

Fixes https://github.com/ToolJet/tj-ee/issues/4393
Fixes https://github.com/ToolJet/tj-ee/issues/4387
Frontend EE - https://github.com/ToolJet/ee-frontend/pull/275
Server EE - https://github.com/ToolJet/ee-server/pull/284

The most important changes are:

License and Feature Access Refactor:

Replaced all direct accesses to license and featureAccess from useStore with the new useLicenseStore hook across multiple components, including ConfigHandle.jsx, RightTopHeaderButtons.jsx, PageMenu.jsx, PagesSidebarNavigation.jsx, PageSettings.jsx, MobileNavigationMenu.jsx, and useAppData.js. This ensures consistent and modular access to license information. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21]

Component Logic Updates:

Updated logic for displaying tooltips and banners related to license validity, including extending license checks to additional component types (e.g., ModuleContainer) and ensuring banners are only shown in appropriate contexts. [1] [2]

Code Cleanup:

Removed unused imports and variables, such as unused services and redundant state selectors, to simplify and clean up the codebase. [1] [2] [3] [4] [5] [6] [7]

Submodule Update:

Updated the frontend/ee submodule to a new commit, likely to pick up related changes in the enterprise edition.

These changes collectively improve maintainability, modularity, and consistency of license-related logic throughout the frontend codebase.

…14299) * feat: enable preview access for view-only users across environments Grant correct permissions for view-only users in app, version, data-source, data-query, and environment modules Redirect view-only users from production to development environment in frontend Align permission checks and environment logic with AppEnvironments.jsx pattern Fix 403 errors on preview, environment switch, and data query run APIs for view-only users * fix: previw links for production url * fix edit links for viewer * update submodule * chore: update version to 3.20.21-lts across all components --------- Co-authored-by: gsmithun4 <gsmithun4@gmail.com>

Co-authored-by: gsmithun4 <3417097+gsmithun4@users.noreply.github.com>

* azure oidc changes * chore: update version to 3.20.22-lts across all components --------- Co-authored-by: gsmithun4 <gsmithun4@gmail.com>

Co-authored-by: gsmithun4 <3417097+gsmithun4@users.noreply.github.com>

…14283)

…le the selected cell exposed variable was not getting updated (#14286)

* Fix-import spec * Minor fixes * Minor fixes * Add data-cy * Update real events dependency

* Hotfix/Azure OIDC Group sync fixes (#14077) * azure oidc changes * chore: update version to 3.20.22-lts across all components --------- Co-authored-by: gsmithun4 <gsmithun4@gmail.com> * 🚀 chore: update submodules to latest lts-3.16 after auto-merge (#14345) Co-authored-by: gsmithun4 <3417097+gsmithun4@users.noreply.github.com> * fix: Page permission tooltip placement issue --------- Co-authored-by: Rohan Lahori <64496391+rohanlahori@users.noreply.github.com> Co-authored-by: gsmithun4 <gsmithun4@gmail.com> Co-authored-by: Adish M <44204658+adishM98@users.noreply.github.com> Co-authored-by: gsmithun4 <3417097+gsmithun4@users.noreply.github.com>

…ement fixed cypress spec for WF's after state management

…4340) * Add cypress test cases for enterprise user groups and permissions * update constants * update user groups test cases * update spec * update groups cases * update lock.json * update custom group cases * update docker file * update first user onboarding and platform commands * add case for UI promote and release permission

…eprecated license check

bito-code-review · 2025-10-23T10:51:25Z

Bito Automatic Review Skipped - Branch Excluded

Bito didn't auto-review because the source or target branch is excluded from automatic reviews.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the branch exclusion settings here, or contact your Bito workspace admin at shubham@tooljet.com.

…remove deprecated license check" This reverts commit b4b5218.

bito-code-review · 2025-10-23T12:25:19Z

Bito Automatic Review Skipped - Branch Excluded

Bito didn't auto-review because the source or target branch is excluded from automatic reviews.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the branch exclusion settings here, or contact your Bito workspace admin at shubham@tooljet.com.

…e related components" This reverts commit cd2936b.

bito-code-review · 2025-10-23T12:31:21Z

Bito Automatic Review Skipped - Branch Excluded

Bito didn't auto-review because the source or target branch is excluded from automatic reviews.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the branch exclusion settings here, or contact your Bito workspace admin at shubham@tooljet.com.

bito-code-review · 2025-10-24T05:33:02Z

Bito Automatic Review Skipped - Branch Excluded

Bito didn't auto-review because the source or target branch is excluded from automatic reviews.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the branch exclusion settings here, or contact your Bito workspace admin at shubham@tooljet.com.

Srimanitejas123 and others added 30 commits October 3, 2025 15:28

resolved conflicts

a564a84

updated spec

ed16583

resolved conflicts

db44bfe

fixed real events

968d95b

resolved conflicts

71b7e3f

updated spec with banner fixes

724e623

🚀 chore: update submodules to latest lts-3.16 after auto-merge (#14333)

5c74cd2

Co-authored-by: gsmithun4 <3417097+gsmithun4@users.noreply.github.com>

added api commands

ef38815

resolved conflicts

fbf79ad

removed imported file

91b5b39

updated spec

3f11a03

updated spec with review comments

956f6be

Fix-import spec

a3b0a33

Minor fixes

4359621

Minor fixes

70be5e6

Add data-cy

0fd25e1

Hotfix/Azure OIDC Group sync fixes (#14077)

27b7aef

* azure oidc changes * chore: update version to 3.20.22-lts across all components --------- Co-authored-by: gsmithun4 <gsmithun4@gmail.com>

🚀 chore: update submodules to latest lts-3.16 after auto-merge (#14345)

274d532

Co-authored-by: gsmithun4 <3417097+gsmithun4@users.noreply.github.com>

resloved conflicts

15b274a

Adding scrollbar to canvas (#14347)

d223893

chore: Fix module being empty on editor (#14341)

46def5e

Update real events dependency

7e17003

fix: Control component actions not working for table action buttons (#…

050dc4f

…14283)

fix: When row is selected in table & we click on action button in tab…

47efc50

…le the selected cell exposed variable was not getting updated (#14286)

Added automation for import export (#14342)

6c3958e

* Fix-import spec * Minor fixes * Minor fixes * Add data-cy * Update real events dependency

cleaned spec with unwanted functions

27d3b89

Merge pull request #14246 from ToolJet/wf_cypressFix_after_statemanag…

a955c7a

…ement fixed cypress spec for WF's after state management

noordeen123 added active-ee-review-app active-ce-review-app create-ee-review-app and removed active-ce-review-app active-ee-review-app create-ee-review-app labels Oct 23, 2025

Merge branch 'lts-3.16' into feat/license-appbuilder-changes

03153bd

kavinvenkatachalam requested review from adishM98, ganesh8056 and slancerk as code owners October 23, 2025 10:45

feat: integrate license validation into Viewer component and remove d…

b4b5218

…eprecated license check

Revert "feat: integrate license validation into Viewer component and …

0b9b449

…remove deprecated license check" This reverts commit b4b5218.

Revert "refactor: migrate license management to a new store and updat…

b941425

…e related components" This reverts commit cd2936b.

chore: update subproject commit reference in server/ee

b2208fe

johnsoncherian merged commit 506ffae into feat/license-flag Oct 24, 2025
17 checks passed

johnsoncherian deleted the feat/license-appbuilder-changes branch October 24, 2025 08:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: License change on AppBuilder#14391

feat: License change on AppBuilder#14391
johnsoncherian merged 49 commits intofeat/license-flagfrom
feat/license-appbuilder-changes

kavinvenkatachalam commented Oct 23, 2025 •

edited

Loading

Uh oh!

bito-code-review bot commented Oct 23, 2025

Uh oh!

bito-code-review bot commented Oct 23, 2025

Uh oh!

bito-code-review bot commented Oct 23, 2025

Uh oh!

bito-code-review bot commented Oct 24, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

Conversation

kavinvenkatachalam commented Oct 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bito-code-review bot commented Oct 23, 2025

Uh oh!

bito-code-review bot commented Oct 23, 2025

Uh oh!

bito-code-review bot commented Oct 23, 2025

Uh oh!

bito-code-review bot commented Oct 24, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

kavinvenkatachalam commented Oct 23, 2025 •

edited

Loading