Page MenuHomeLubuntu Development

Changed connection editor to nm-connection-editor
ClosedPublic

Authored by hmollercl on Mon, Jul 1, 9:31 AM.

Details

Summary

Changed connection editor to nm-connection-editor from network-manager-gnome

Test Plan

check if nm-tray open nm-connection-editor instead of nmtui

Diff Detail

Repository
rNMTRAYPACKAGING nm-tray Packaging
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

hmollercl requested review of this revision.Mon, Jul 1, 9:31 AM
hmollercl created this revision.
Harbormaster completed remote builds in B30: Diff 43.Mon, Jul 1, 9:31 AM
Harbormaster completed remote builds in B30: Diff 43.
tsimonq2 requested changes to this revision.Mon, Jul 1, 10:21 AM

Fairly nitpicky, but your modifications to the patch completely change the purpose. That's fine, but you can now just modify existing information Description, Author, and Bug instead of adding your own lines. Also, please modify Bug to say Bug-Ubuntu, which follows DEP-3.

Otherwise, LGTM. Thanks!

This revision now requires changes to proceed.Mon, Jul 1, 10:21 AM
hmollercl updated this revision to Diff 47.Tue, Jul 2, 9:14 AM

update description of patch

Harbormaster completed remote builds in B34: Diff 47.Tue, Jul 2, 9:14 AM
Harbormaster completed remote builds in B34: Diff 47.
hmollercl updated this revision to Diff 48.Tue, Jul 2, 9:17 AM

updated patch bug, and authot

Harbormaster completed remote builds in B35: Diff 48.Tue, Jul 2, 9:17 AM
Harbormaster completed remote builds in B35: Diff 48.
tsimonq2 requested changes to this revision.Tue, Jul 2, 9:42 AM

network-manager-gnome, not network-manager-gnomer :)

Otherwise, LGTM

This revision now requires changes to proceed.Tue, Jul 2, 9:42 AM
hmollercl updated this revision to Diff 50.Wed, Jul 3, 7:54 PM

correct typo

Harbormaster completed remote builds in B37: Diff 50.Wed, Jul 3, 7:54 PM
tsimonq2 accepted this revision.Wed, Jul 3, 9:16 PM
This revision is now accepted and ready to land.Wed, Jul 3, 9:16 PM
wxl accepted this revision.Thu, Jul 4, 2:09 PM
This revision was automatically updated to reflect the committed changes.