-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add direct download option to users in register template form #10193
base: main
Are you sure you want to change the base?
Add direct download option to users in register template form #10193
Conversation
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.
code lgtm
@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10193 +/- ##
=========================================
Coverage 16.16% 16.16%
Complexity 13015 13015
=========================================
Files 5644 5644
Lines 494640 494640
Branches 59937 59937
=========================================
Hits 79955 79955
Misses 405858 405858
Partials 8827 8827
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
UI build: ✔️ |
@lucas-a-martins , it does not seem to work in qa. I can not tell from the API if this needs a version after 4.20. Do you know? |
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.
LGTM - haven't tested it though
@DaanHoogland , I tried to test in the same QA (#10193 (comment)), but the only hypervisor option it has for normal users is "Simulator". Even before this PR, the direct download only works if the hypervisor is KVM or |
Description
Normal users are allowed to register a direct download template via API, but the option is disabled in the UI.
This PR fix the issue #9923 by removing this restriction.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
The template registered by a normal user using the direct download option:
How Has This Been Tested?
I tested by registering a template via UI using an account with the User role. Selecting the KVM hypervisor gave me the direct download option.