-
Notifications
You must be signed in to change notification settings - Fork 261
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
[2.0] Move port list to server #676
Comments
I'm on this task to fix it in priority because could break a lot of stuff. |
I will take advantage of this task for implementing port update in the GUI instead of clear all and replace in some nodes. |
julien-duponchelle
added a commit
that referenced
this issue
Sep 13, 2016
julien-duponchelle
added a commit
that referenced
this issue
Sep 14, 2016
julien-duponchelle
added a commit
to GNS3/gns3-gui
that referenced
this issue
Sep 14, 2016
julien-duponchelle
added a commit
that referenced
this issue
Sep 14, 2016
Ok it's done I hope to not introduce bugs. It seem to fix several gui bugs due to issue of sync with server. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Actually the list of port is computed by the GUI but this mean we need to rewrite everything for the futur web application.
Instead of duplicating all the code we will move everything to a central location in the controller. Making easier to improve port name formating, check of what kind of port could be connected together...
This will help for #654 and GNS3/gns3-gui#1502
The text was updated successfully, but these errors were encountered: