Make update page labels consistent with topUpdateComp
ClosedPublic

Authored by MrPepe on Tue, Aug 20, 2:20 PM.

Details

Summary

Made the labels on the update page consistent with the ones shown in the topUpdateComp of the main window

Test Plan

Check for updates and see that the labels have changed

Diff Detail

Repository
R134 Discover Software Store
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
MrPepe created this revision.Tue, Aug 20, 2:20 PM
Restricted Application added a project: Plasma. · View Herald TranscriptTue, Aug 20, 2:20 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
MrPepe requested review of this revision.Tue, Aug 20, 2:20 PM
ngraham accepted this revision.Tue, Aug 20, 2:22 PM
ngraham added a reviewer: Discover Software Store.
ngraham added a subscriber: ngraham.

How serindipitous; I was just about to do this myself! Needless to say, LGTM.

This revision is now accepted and ready to land.Tue, Aug 20, 2:23 PM

This was my first contribution so I was following the wiki guide.
Running ctest showed that knsbackendtest fails. Is that a setup problem on my side or something that actually does not work?

apol added a subscriber: apol.Tue, Aug 20, 3:43 PM

Thanks. The problem with the knsbackendtest is most likely unrelated. You can ignore it here.

Do you have permissions to land the patch?

@MrPepe I cannot land this patch because our commit hookscript requires a last name in addition to a first name. Can you tell me your last name and adjust your .gitconfig file to include that in future patches?

Thanks!

This revision was automatically updated to reflect the committed changes.

Figured it out from your email address. :) Do please update your .gitconfig file for future patches though!

Weird, I actually updated my git config before comitting. Will pay attention to it next time.
Thanks for landing it!