Skip to content

feat!(ui): integrate Pinia store and enhance enterprise paywall #4708

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

luannmoreira
Copy link
Member

Description

This PR introduces several improvements and fixes in the UI, including:

  • State Management Enhancement: Added Pinia as the new state management solution.
  • Global Components Registration: Automatically registers global components to simplify usage.
  • Enterprise Upgrade Banner: Displays an upgrade banner and paywall prompts for non-enterprise users.
  • Dependency Updates: Upgraded @vue/devtools-kit and @vue/devtools-shared to 7.7.2.
  • Code Cleanup: Removed PaywallDialog.vue as it's now handled differently.

Changes

Core Enhancements

  • Introduced Pinia as a state management solution.

  • Registered global components dynamically in main.ts.

  • Enterprise Features & Paywall

  • Added an Enterprise upgrade banner to the login page (Login.vue).

  • Disabled input fields and login button for non-enterprise users.

  • Replaced PaywallDialog.vue with a more modular approach.

  • Dependency Updates

  • Updated @vue/devtools-kit & @vue/devtools-shared from 7.5.2 → 7.7.2.

  • Installed pinia v3.0.1 as a new dependency.

  • Code Cleanup & Removals

  • Removed PaywallDialog.vue

  • Refactored UserWarning.vue to use the new PaywallDialog logic.

How to Test

  1. Run npm install to install pinia.
  2. Start the application and navigate to /login.
  3. Verify the paywall banner & disabled fields for non-enterprise users.
  4. Ensure that existing functionalities (store, routing) work correctly.

@luannmoreira luannmoreira added area/ui do-not-merge status/needs-review javascript Pull requests that update Javascript code product/cloud Issue/PR related to Cloud Edition release/breaking-change kind/refactor product/enterprise Issue/PR related to Enterprise Edition status/under-testing Tests broadened labels Apr 3, 2025
@luannmoreira luannmoreira self-assigned this Apr 3, 2025
@luannmoreira luannmoreira requested a review from a team as a code owner April 3, 2025 13:06
@luannmoreira luannmoreira force-pushed the feat/pinia-global-paywall branch from 7c4abe4 to b9be8ce Compare April 3, 2025 13:07
- Added Pinia store integration in `main.ts` for state management.
- Registered global components dynamically from `@global/components/index`.
- Implemented an enterprise paywall banner and upgrade button in `Login.vue`.
- Disabled form inputs and login button for non-enterprise users.
- Replaced `PaywallDialog.vue` component with a global import.
- Upgraded Vue DevTools dependencies to `7.7.2` and added `pinia` to
`package.json`.

BREAKING CHANGE: `PaywallDialog.vue` has been removed and must be referenced
globally.
@luannmoreira luannmoreira force-pushed the feat/pinia-global-paywall branch from b9be8ce to 2d865c6 Compare April 3, 2025 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui do-not-merge javascript Pull requests that update Javascript code kind/refactor product/cloud Issue/PR related to Cloud Edition product/enterprise Issue/PR related to Enterprise Edition release/breaking-change status/needs-review status/under-testing Tests broadened
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants