Skip to content

Conversation

@nikhilsaikethe
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the ☢️ Bug Something isn't working label Dec 4, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

Failed to generate code suggestions for PR

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 4, 2025

Greptile Overview

Greptile Summary

This PR adds a "Manage License" button to the About page's Enterprise License Details section. The button navigates users to the license management page after switching to the meta organization.

  • Added navigateToLicense function that retrieves the meta org from config, validates user access, switches organization context, and routes to the license page
  • Includes authorization check that displays an error notification if the user doesn't have access to the meta organization
  • Added corresponding translation key manage_license in the English locale file
  • Button only appears for enterprise non-cloud deployments (consistent with existing Enterprise License Details section visibility)

Confidence Score: 4/5

  • This PR is safe to merge with low risk - it adds UI navigation functionality with proper authorization checks
  • Score reflects straightforward feature addition with proper error handling. Minor pattern inconsistency with not calling useLocalOrganization before store dispatch, but this doesn't affect functionality since URL params are used for org selection on page load.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
web/src/locales/languages/en.json 5/5 Added new translation key manage_license for the license management button label
web/src/views/About.vue 4/5 Added "Manage License" button with navigation logic that switches to meta org before routing to license page; includes authorization check with user notification

Sequence Diagram

sequenceDiagram
    participant User
    participant AboutPage as About.vue
    participant Store as Vuex Store
    participant Router
    participant LicensePage

    User->>AboutPage: Click "Manage License" button
    AboutPage->>Store: Get meta_org from zoConfig
    AboutPage->>Store: Find metaOrg in organizations list
    
    alt User has access to meta org
        AboutPage->>Store: dispatch("setSelectedOrganization", metaOrgOption)
        Store-->>AboutPage: Organization updated
        AboutPage->>Router: push({ name: 'license', query: { org_identifier } })
        Router->>LicensePage: Navigate to license page
    else User lacks access
        AboutPage->>User: Show error notification "You are not authorized to manage the license"
    end
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

nikhilsaikethe and others added 2 commits December 4, 2025 19:32
Auto-generated translation updates from English source file.

🤖 Generated with automated translation workflow
@bjp232004 bjp232004 merged commit 8f3ab5c into main Dec 4, 2025
38 checks passed
@bjp232004 bjp232004 deleted the fix/issue-9478 branch December 4, 2025 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

☢️ Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants