-
Notifications
You must be signed in to change notification settings - Fork 406
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
feat: Adds Licensing upload tab #4934
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Docker builds report
|
Uffizzi Ephemeral Environment
|
Signed-off-by: Tiago Fonseca <[email protected]>
Signed-off-by: Tiago Fonseca <[email protected]>
As far as I can tell, it's not possible (or at least difficult) to test this PR without this code. |
@@ -237,6 +242,9 @@ const OrganisationSettingsPage = class extends Component { | |||
...[ | |||
SettingsTab.General, | |||
paymentsEnabled && !isAWS ? SettingsTab.Billing : null, | |||
isEnterprise && isSaas === false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor but cant this just be isEnterprise? I'd have thought that would make saas always false
Thanks for submitting a PR! Please check the boxes below:
docs/
if required so people know about the feature!Changes
Reference: #4923
This pull request introduces a new feature for uploading organization licensing files. The changes primarily focus on adding new endpoints, updating types, and integrating the new functionality into the existing UI components.
focus
method.How did you test this code?
Tested mannualy