feat: implement upgrade button within editor header by jacquesikot · Pull Request #41124 · appsmithorg/appsmith

jacquesikot · 2025-07-22T05:58:57Z

Summary

This PR adds an upgrade button to the IDE header, allowing users to access upgrade options directly from the editor interface.

Changes Made

Added ShowUpgradeMenuItem import from ee/utils/licenseHelpers
Integrated <ShowUpgradeMenuItem /> component into the IDE header right section
Positioned the upgrade menu item before the help bar for better visibility

Automation

/ok-to-test tags="@tag.Sanity, @tag.IDE"

🔍 Cypress test results

Tip

🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/16440833655
Commit: e5fcc5f
Cypress dashboard.
Tags: @tag.Sanity, @tag.IDE
Spec:

Tue, 22 Jul 2025 10:38:16 UTC

Communication

Should the DevRel and Marketing teams inform users about this change?

Yes
No

Summary by CodeRabbit

New Features
- Added an upgrade menu item to the IDE header for easier access to upgrade options.

coderabbitai · 2025-07-22T05:59:04Z

Walkthrough

A new ShowUpgradeMenuItem component is imported and added to the right section of the IDE header, positioned before the existing HelpBar. No other logic, state, or control flow is affected by this change.

Changes

File(s)	Change Summary
app/client/src/pages/AppIDE/layouts/components/Header/index.tsx	Imported and rendered `ShowUpgradeMenuItem` in the IDE header before `HelpBar`.

Estimated code review effort

1 (~2 minutes)

Poem

In the header’s gentle light,
A new menu takes its flight—
Upgrade beckons, shining bright,
Just before Help’s guiding sight.
Code grows with every line,
In this IDE, refined by design!
✨🛠️

📜 Recent review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 11a5a96 and e5fcc5f.

📒 Files selected for processing (1)

app/client/src/pages/AppIDE/layouts/components/Header/index.tsx (2 hunks)

🧠 Learnings (2)

📓 Common learnings

Learnt from: Aishwarya-U-R
PR: appsmithorg/appsmith#29405
File: app/client/cypress/e2e/Regression/ClientSide/Binding/TableV2_Widget_API_Pagination_spec.js:37-41
Timestamp: 2024-07-26T21:12:57.228Z
Learning: The pull request titled "test: Cypress | Replace static with Dynamic waits - Part 1" is part of a phased approach where only certain test specifications are targeted for static wait removal in the initial phase. Future phases will address additional specs.

Learnt from: Aishwarya-U-R
PR: appsmithorg/appsmith#29405
File: app/client/cypress/e2e/Regression/ClientSide/Binding/TableV2_Widget_API_Pagination_spec.js:37-41
Timestamp: 2024-10-08T15:32:39.374Z
Learning: The pull request titled "test: Cypress | Replace static with Dynamic waits - Part 1" is part of a phased approach where only certain test specifications are targeted for static wait removal in the initial phase. Future phases will address additional specs.

Learnt from: sharat87
PR: appsmithorg/appsmith#30252
File: deploy/docker/fs/usr/lib/python3/dist-packages/supervisor/appsmith_supervisor_stdout.py:21-29
Timestamp: 2024-07-26T21:12:57.228Z
Learning: The user has confirmed that the suggested changes to handle potential exceptions and improve the robustness of the `main` function in `appsmith_supervisor_stdout.py` are acceptable.

Learnt from: KelvinOm
PR: appsmithorg/appsmith#29387
File: app/client/packages/design-system/widgets/src/components/TagGroup/src/Tag.tsx:9-9
Timestamp: 2024-10-08T15:32:34.114Z
Learning: The `CloseIcon` is being moved to a common directory for better reusability across components, following the suggestion to avoid importing it from the `Modal` component's directory.

Learnt from: KelvinOm
PR: appsmithorg/appsmith#29387
File: app/client/packages/design-system/widgets/src/components/TagGroup/src/Tag.tsx:9-9
Timestamp: 2024-07-26T21:12:57.228Z
Learning: The `CloseIcon` is being moved to a common directory for better reusability across components, following the suggestion to avoid importing it from the `Modal` component's directory.

app/client/src/pages/AppIDE/layouts/components/Header/index.tsx (9)

Learnt from: KelvinOm
PR: #29387
File: app/client/packages/design-system/widgets/src/components/TagGroup/src/Tag.tsx:9-9
Timestamp: 2024-10-08T15:32:34.114Z
Learning: The CloseIcon is being moved to a common directory for better reusability across components, following the suggestion to avoid importing it from the Modal component's directory.

Learnt from: KelvinOm
PR: #29387
File: app/client/packages/design-system/widgets/src/components/TagGroup/src/Tag.tsx:9-9
Timestamp: 2024-07-26T21:12:57.228Z
Learning: The CloseIcon is being moved to a common directory for better reusability across components, following the suggestion to avoid importing it from the Modal component's directory.

Learnt from: ashit-rath
PR: #37912
File: app/client/src/git/components/QuickActions/helpers.ts:22-25
Timestamp: 2024-12-03T10:13:43.282Z
Learning: In app/client/src/git/components/QuickActions/helpers.ts, the unnecessary @ts-ignore comments will be removed in future PRs.

Learnt from: ashit-rath
PR: #38098
File: app/client/src/git/components/GitQuickActions/index.test.tsx:18-19
Timestamp: 2024-12-11T11:42:51.143Z
Learning: In app/client/src/git/components/GitQuickActions, the Statusbar component is located in the parent directory, so it should be imported and mocked using "../Statusbar".

Learnt from: brayn003
PR: #38171
File: app/client/src/git/components/DefaultBranch/DefaultBranchCE.tsx:1-14
Timestamp: 2024-12-15T17:45:48.303Z
Learning: In app/client/src/git/components/DefaultBranch/DefaultBranchCE.tsx, the feature flag check is performed at a higher level, so it's acceptable to have isGitProtectedFeatureLicensed={false} in this component.

Learnt from: brayn003
PR: #38088
File: app/client/src/git/components/GitContextProvider/hooks/useGitBranches.ts:40-43
Timestamp: 2024-12-11T08:25:39.197Z
Learning: In app/client/src/git/components/GitContextProvider/hooks/useGitBranches.ts, the useMemo hook includes dependencies artifactType and baseArtifactId in its dependency array.

Learnt from: ankitakinger
PR: #37330
File: app/client/src/pages/common/SearchBar/HomepageHeaderAction.tsx:95-95
Timestamp: 2024-11-12T11:42:28.998Z
Learning: In icon provider components within the TypeScript/React codebase, old settings icons like "settings-2-line" and "settings-control" are intentionally provided alongside new icons. These references are acceptable and should not be flagged for updates.

Learnt from: brayn003
PR: #38088
File: app/client/src/git/components/GitQuickActions/BranchButton/index.tsx:72-74
Timestamp: 2024-12-11T08:33:24.352Z
Learning: In the 'BranchButton' component in 'app/client/src/git/components/GitQuickActions/BranchButton/index.tsx' (TypeScript, React), the useEffect hook that checks for label ellipsis does not need to include currentBranch in its dependency array.

Learnt from: brayn003
PR: #38088
File: app/client/src/git/components/GitQuickActions/helpers/getPullButtonStatus.ts:16-21
Timestamp: 2024-12-11T08:31:10.356Z
Learning: In the file app/client/src/git/components/GitQuickActions/helpers/getPullButtonStatus.ts, when defining the function getPullBtnStatus, it's acceptable to use default parameters in destructuring. Do not suggest removing default parameters in this context.

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: Aishwarya-U-R
PR: appsmithorg/appsmith#29405
File: app/client/cypress/e2e/Regression/ClientSide/Binding/TableV2_Widget_API_Pagination_spec.js:37-41
Timestamp: 2024-07-26T21:12:57.228Z
Learning: The pull request titled "test: Cypress | Replace static with Dynamic waits - Part 1" is part of a phased approach where only certain test specifications are targeted for static wait removal in the initial phase. Future phases will address additional specs.

Learnt from: Aishwarya-U-R
PR: appsmithorg/appsmith#29405
File: app/client/cypress/e2e/Regression/ClientSide/Binding/TableV2_Widget_API_Pagination_spec.js:37-41
Timestamp: 2024-10-08T15:32:39.374Z
Learning: The pull request titled "test: Cypress | Replace static with Dynamic waits - Part 1" is part of a phased approach where only certain test specifications are targeted for static wait removal in the initial phase. Future phases will address additional specs.

Learnt from: sharat87
PR: appsmithorg/appsmith#30252
File: deploy/docker/fs/usr/lib/python3/dist-packages/supervisor/appsmith_supervisor_stdout.py:21-29
Timestamp: 2024-07-26T21:12:57.228Z
Learning: The user has confirmed that the suggested changes to handle potential exceptions and improve the robustness of the `main` function in `appsmith_supervisor_stdout.py` are acceptable.

Learnt from: KelvinOm
PR: appsmithorg/appsmith#29387
File: app/client/packages/design-system/widgets/src/components/TagGroup/src/Tag.tsx:9-9
Timestamp: 2024-10-08T15:32:34.114Z
Learning: The `CloseIcon` is being moved to a common directory for better reusability across components, following the suggestion to avoid importing it from the `Modal` component's directory.

Learnt from: KelvinOm
PR: appsmithorg/appsmith#29387
File: app/client/packages/design-system/widgets/src/components/TagGroup/src/Tag.tsx:9-9
Timestamp: 2024-07-26T21:12:57.228Z
Learning: The `CloseIcon` is being moved to a common directory for better reusability across components, following the suggestion to avoid importing it from the `Modal` component's directory.