Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes proposed in this pull request
Related issue(s)
Copilot generated summary
Provide a Copilot generated summary of the changes in this pull request.
Copilot summary
This pull request introduces several changes to enhance code readability and functionality across multiple files in the
NETworkManagerproject. The most significant updates include replacing theGetGroupmethod withGetGroupByNamefor improved clarity, refactoring dialog management for group operations, and reorderingusingdirectives for consistency.Method Updates for Group Retrieval:
ProfileManager.GetGroupwithProfileManager.GetGroupByNamein multipleCreateSessionInfomethods across files such asAWSSessionManager.cs,PowerShell.cs,PuTTY.cs,RemoteDesktop.cs,SNMP.cs, andTigerVNC.cs. This change improves clarity by explicitly indicating the retrieval is based on the group's name. [1] [2] [3] [4] [5] [6]Dialog Management Refactor:
ProfileDialogManager.cs. The refactor replacesMetroDialogwithChildWindowfor better integration with the application's window hierarchy.ProfileDialogManager.ShowEditGroupDialogin various view models (AWSSessionManagerHostViewModel.cs,DNSLookupHostViewModel.cs,IPGeolocationHostViewModel.cs,IPScannerHostViewModel.cs,NetworkInterfaceViewModel.cs,PingMonitorHostViewModel.cs) to useApplication.Current.MainWindowfor dialog invocation. [1] [2] [3] [4] [5] [6]Code Cleanup:
usingdirectives across several files for improved readability and consistency. [1] [2] [3] [4] [5] [6]These changes collectively enhance code clarity, maintainability, and user interface consistency in the
NETworkManagerproject.To-Do
Contributing
By submitting this pull request, I confirm the following: