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
Fixes #6912/BZ 1126446 - Sync Plan enabled by default, #4651
Fixes #6912/BZ 1126446 - Sync Plan enabled by default, #4651
Conversation
5fd17cb
to
33f20ba
Compare
@@ -29,7 +29,7 @@ angular.module('Bastion.sync-plans').controller('SyncPlanDetailsInfoController', | |||
function ($scope, $q, translate, SyncPlan, MenuExpander) { | |||
$scope.successMessages = []; | |||
$scope.errorMessages = []; | |||
$scope.intervals = ['none', 'hourly', 'daily', 'weekly']; | |||
$scope.intervals = [translate('hourly'), translate('daily'), translate('weekly')]; |
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.
I don't think you want to mark these for translation as they need to match up with these: https://github.com/Katello/katello/pull/4651/files#diff-cbbf4a31420b097c086fac10249ad959L25
Poor design to use a string that is then displayed to the user. In order to get these translated we could use a filter, but I am fine doing that in another PR. For this PR, please just revert this line.
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.
I did not realize, it should be fixed now.
33f20ba
to
829a49d
Compare
@@ -1478,3 +1478,7 @@ ul { | |||
list-style: none; | |||
margin: 0; | |||
} | |||
|
|||
.gray{ |
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.
Is this used anywhere?
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.
When sync plan is disabled, interval on details page is in gray
…x to disable on edit
829a49d
to
9291c47
Compare
Error message should be fixed now |
ACK |
Fixes #6912/BZ 1126446 - Sync Plan enabled by default,
checkbox to disable on edit