-
Notifications
You must be signed in to change notification settings - Fork 35
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
Feature/disable apikey #1518
Feature/disable apikey #1518
Conversation
Blocked by #1520 |
Status update? Can you start by adding UI and only then store to UI? |
@bajiat, yes, the problem is that I couldn't submit Proxy form. Trying it again today. |
'disable_api_key': { | ||
type: Boolean, | ||
optional: true, | ||
label: 'Disable API Key', |
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.
We need to internationalize this label.
Add internationalization. |
This is really close. Just switch identifiers to ID rather than class, when there is only one 'instance' or a particular element on a page. |
@@ -120,9 +120,31 @@ | |||
{{_ "proxyBackendForm_apiPort_helpText" }} | |||
</p> | |||
</div> | |||
|
|||
<!-- Advanced settings --> | |||
<div class="form-group col-md-12 advanced-settings"> |
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.
If this is the only 'advanced-settings' element on the page, use id
instead of class
.
</div> | ||
|
||
<!-- submit button --> | ||
<button type="submit" class="btn btn-primary pull-right">Save proxy settings</button> | ||
<button type="submit" class="btn btn-success pull-left save-proxy-button"> |
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.
If this is the only 'save-proxy-button' element on the page, use id
rather than class
.
Closes #622