Skip to content
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

[Feat] Data catalog: distinguish label and name of nodes #780

Closed
petrasovaa opened this issue Jul 11, 2020 · 0 comments · Fixed by #783
Closed

[Feat] Data catalog: distinguish label and name of nodes #780

petrasovaa opened this issue Jul 11, 2020 · 0 comments · Fixed by #783
Labels
enhancement New feature or request gsoc Reserved for Google Summer of Code student(s) GUI wxGUI related

Comments

@petrasovaa
Copy link
Contributor

Currently, node label is used for most management tasks, e.g. when switching mapset it takes the mapset name from the label (node.label). Instead it should take the name from the data attribute of node (node.data['name']) when appropriate. This way we can use the label to show additional information about the node in the GUI, e.g. number of maps in the mapset, if mapset is locked etc. So the task here is just to review all usages of label and replace it with data['name] if it needs the name of the map/mapset/location/db.

@petrasovaa petrasovaa added enhancement New feature or request GUI wxGUI related gsoc Reserved for Google Summer of Code student(s) labels Jul 11, 2020
@petrasovaa petrasovaa added this to To do in New GUI Startup via automation Jul 11, 2020
@petrasovaa petrasovaa linked a pull request Jul 13, 2020 that will close this issue
@petrasovaa petrasovaa moved this from To do to In progress in New GUI Startup Jul 14, 2020
New GUI Startup automation moved this from In progress to Done Jul 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gsoc Reserved for Google Summer of Code student(s) GUI wxGUI related
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant