fix: license fixes sync to main #9352
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bjp232004
approved these changes
Nov 27, 2025
Contributor
|
Failed to generate code suggestions for PR |
Contributor
Greptile OverviewGreptile SummaryThis PR consolidates three license-related fixes: disabling license data loading for cloud environments in About page, displaying default ingestion limits when licenses expire, and simplifying meta org detection to use only identifier matching.
Confidence Score: 4/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant User
participant AboutPage as About.vue
participant LicenseComponent as License.vue
participant useIsMetaOrg
participant Store as Vuex Store
participant LicenseAPI
User->>AboutPage: Load About page
AboutPage->>AboutPage: Check config.isCloud
alt config.isCloud == 'false' (Self-hosted)
AboutPage->>LicenseAPI: loadLicenseData()
LicenseAPI-->>AboutPage: License data
AboutPage->>AboutPage: Display license info with limits
alt license.expired
AboutPage->>AboutPage: Show default limits (100 GB/day, PerDayCount)
else license valid
AboutPage->>AboutPage: Show actual license limits
end
else config.isCloud == 'true' (Cloud)
AboutPage->>AboutPage: Skip license data loading
end
User->>LicenseComponent: Access License tab in Settings
LicenseComponent->>useIsMetaOrg: Check isMetaOrg
useIsMetaOrg->>Store: Get selectedOrganization.identifier
useIsMetaOrg->>Store: Get zoConfig.meta_org
useIsMetaOrg-->>LicenseComponent: identifier === meta_org
alt isMetaOrg && isEnterprise
LicenseComponent->>LicenseComponent: Show License tab
LicenseComponent->>LicenseAPI: Load license data
LicenseAPI-->>LicenseComponent: License data
alt license.expired
LicenseComponent->>LicenseComponent: Show default limits (100 GB/day, PerDayCount)
else license valid
LicenseComponent->>LicenseComponent: Show actual license limits
end
else not meta org
LicenseComponent->>LicenseComponent: Hide License tab
end
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
nikhilsaikethe
force-pushed
the
fix/hide-alert-history
branch
9 times, most recently
from
a9b0832 to
000fc27
Compare
nikhilsaikethe
force-pushed
the
fix/hide-alert-history
branch
from
000fc27 to
ea029f7
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
don't load license info for cloud in about us page
show license section for meta orgs in cloud as well
show default ingestion limit when license expired