Univention Bugzilla – Bug 33190
Release policy layout
Last modified: 2015-02-24 08:50:25 CET
Created attachment 5574 [details] umc-policy-release.png See screenshot, the layout should be adjusted.
IMHO the checkbox to enable/disable the policy should be removed, as this is the only UCS policy providing such as mis-guided option. Everywhere else the policy has to be removed to un-apply the policy. This check-box already has caused confusion in UCS technical trainings.
(In reply to Philipp Hahn from comment #1) > IMHO the checkbox to enable/disable the policy should be removed, as this is > the only UCS policy providing such as mis-guided option. Everywhere else the > policy has to be removed to un-apply the policy. > This check-box already has caused confusion in UCS technical trainings. Ack,there's already Bug 20962 for this, which was also suggested during a training.
At least in former times the updater updated to the latest version if the field release version was not set. IF the checkbox will be removed this behavior should be changed as well, otherwise all systems will be performing the update. UCS mixed scenarios should also be considered.
Created attachment 5973 [details] screenshot: adjusted umc-policy-release
Created attachment 5974 [details] patch adjusts the layout
(In reply to Stefan Gohmann from comment #0) > Created attachment 5574 [details] > umc-policy-release.png > > See screenshot, the layout should be adjusted. I only adjusted the layout. Did not touch Bug 20962
patch assigned
(In reply to Philipp Hahn from comment #1) > IMHO the checkbox to enable/disable the policy should be removed, as this is > the only UCS policy providing such as mis-guided option. Everywhere else the > policy has to be removed to un-apply the policy. > This check-box already has caused confusion in UCS technical trainings. Actually the text for the checkbox is misleading, as the checkbox toggles between two modes: - checked an release update is performed, if one is available. The text box can be used to optionally specify the maximum version to upgrade to. - unchecked only pending packages are installed, for example errata updates. - without a policy no package or release updates are performed. IMHO the text should be changed as well to better reflect that and the checkbox must be located before the text entry field, to describe the dependency between them.
Created attachment 5996 [details] patch adjusts the layout and adapts some descriptions Layout: The checkbox is now located over the text entry field Descriptions: Adapted misleading texts (old / new) # Policy: Release / Policy: Update # Release / Automatic updates The policy is also for errata-updates so it is misleading to call it 'Release' # Activate policy / Activate release updates. (Errata updates are activated by default) This is what the code does (see: base/univention-updater/debian/univention-updater.univention-maintenance.init) # Release Version / Highest release version to be installed. Also added a long_description: Without specifying the most recent version will be used
Alex, could you please add a little screenshot of how it looks like with the patch? Thanks :) .
Created attachment 6019 [details] adapted policy ger and eng preview
Created attachment 6020 [details] patch adjusts the layout and adapts some descriptions Adapted the patch
Created attachment 6021 [details] updated preview a preview for the renamed 'release policy'
(In reply to Alexander Kramer from comment #13) > Created attachment 6021 [details] > updated preview > > a preview for the renamed 'release policy' Looks good! Suggestion for text change: "Highest release version to be installed" [en] → "Update up to this UCS version" [de] → "Bis zu dieser UCS-Version aktualisieren" It would be nice to have the release version field grayed out if the check box is disabled. This is not possible in a generic way, however, it would be possible to add into DetailPage.js (in univention-management-console-module-udm) an if-case that implements this dynamic handling for this specific policy type. Could you have a try to implement this behaviour? If you have questions regarding the implementation, I am sure Dirk is happy to help :) .
(In reply to Alexander Kläser from comment #14) > Looks good! Suggestion for text change: > > "Highest release version to be installed" > [en] → "Update up to this UCS version" > [de] → "Bis zu dieser UCS-Version aktualisieren" > I adopted the suggestions. See the new patch. For the adaption of the de.po i used this instructions from the wiki: https://hutten.knut.univention.de/mediawiki/index.php/Uebersetzung It worked finde but i am a little bit unsure, if i missed something. > It would be nice to have the release version field grayed out if the check > box is disabled. This is not possible in a generic way, however, it would be > possible to add into DetailPage.js (in > univention-management-console-module-udm) an if-case that implements this > dynamic handling for this specific policy type. > > Could you have a try to implement this behaviour? If you have questions > regarding the implementation, I am sure Dirk is happy to help :) . For this part i would appreciate your help. I spoke with Dirk and Florian about this and both said that this is maybe no good idea.
Created attachment 6089 [details] New patch from 2014-09-04
*** Bug 36699 has been marked as a duplicate of this bug. ***
Patch looks good, please apply to UCS4 errata scope.
Revision 56793 univention-directory-manager-modules (10.0.29-10) * Bug #33190: Changed the release policy to update policy and adapted the layout Successful build Package: univention-directory-manager-modules Version: 10.0.29-10.1272.201412150955 r56796 yaml file
There is one untranslated string: + long_description='Without specifying the most recent version will be used',
r 56809, 56810: Added missing tranlation and adpated the changelog Successful build Package: univention-directory-manager-modules Version: 10.0.29-11.1273.201412151212 r 56811 yaml file
Please make the following text adjustments Activate release updates. (Errata updates are activated by default) → Activate release updates (Errata updates are activated by default). Aktiviere Release-Updates. (Errata Updates sind per Default aktiviert) → Aktiviere Release-Updates (Errata-Updates sind standardmäßig aktiviert). The title differs in eglishn/german: Richtlinie: Updates (german) Policy: Update (english) And when opening the policy then there is a completely new title: Automatische Updates (german) Automatic updates (english) I think this should be changed, too.
I fixed the things from comment #22 in svn r57372.
(In reply to Florian Best from comment #23) > I fixed the things from comment #22 in svn r57372. YAML file: OK, minor adjustments [r57402] Changes: OK Package version: OK
<http://errata.univention.de/ucs/4.0/43.html>