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

show Kolibri IP and port in the web UI #1910

Closed
indirectlylit opened this issue Jul 27, 2017 · 6 comments · Fixed by #2674
Closed

show Kolibri IP and port in the web UI #1910

indirectlylit opened this issue Jul 27, 2017 · 6 comments · Fixed by #2674
Assignees
Labels
P0 - critical Priority: Release blocker or regression TAG: new feature New user-facing feature TAG: user strings Application text that gets translated
Milestone

Comments

@indirectlylit
Copy link
Contributor

During a discussion about native desktop apps and installers, it came up that administrators can often use an easy way to get the current IP and port through the web UI (e.g. Manage -> Device?).

This would be particularly helpful for users that don't use the terminal, e.g. windows users running the tray application.

see related issue #1595

cc @jtamiace would need design

@indirectlylit indirectlylit added the TAG: new feature New user-facing feature label Jul 27, 2017
@indirectlylit indirectlylit added this to the 0.7.0 milestone Aug 11, 2017
@indirectlylit indirectlylit added the P0 - critical Priority: Release blocker or regression label Oct 10, 2017
@jtamiace
Copy link
Member

jtamiace commented Oct 25, 2017

Approved in the design room on Slack.

Solution is to add another "Info" tab in Device with the IP, port, and other useful metadata about the device.

Note for the implementor: if any items are difficult to access through python, skip for now. Priority is getting the URL to show correctly

device info tab

Addendum by @rtibbles - should be visible to any user who is able to view the Device app.

@indirectlylit
Copy link
Contributor Author

note from @jamalex:

We can get all of that from Python except device name/id (unless we use the "instance id" from Morango, which might actually be good, noting however that it can sometimes change), and probably memory usage. In the disk space section, I guess we would just show the free space in the content folder, specifically. Perhaps put that in a section next to the path to the content folder.

@indirectlylit indirectlylit added DEV: frontend TODO: needs gherkin update Add to our manual integration tests P1 - important Priority: High impact on UX and removed P0 - critical Priority: Release blocker or regression labels Oct 27, 2017
@rtibbles rtibbles self-assigned this Nov 1, 2017
@rtibbles
Copy link
Member

rtibbles commented Nov 1, 2017

Taking on the backend portion of this. Will create an API endpoint that exposes the information listed in the mockup above.

@rtibbles
Copy link
Member

rtibbles commented Nov 1, 2017

Note that we can only display a database path if the server is using SQLite (which will normally be the case, but we do support other DB backends).

@rtibbles
Copy link
Member

rtibbles commented Nov 1, 2017

Note to @jamalex: platform.node() seems to work to return a device name.

@rtibbles
Copy link
Member

rtibbles commented Nov 1, 2017

We could get free memory, but it would require us to add the psutil library which we have had significant issues with previously.

@rtibbles rtibbles mentioned this issue Nov 1, 2017
9 tasks
@indirectlylit indirectlylit added the TAG: user strings Application text that gets translated label Nov 15, 2017
@indirectlylit indirectlylit self-assigned this Nov 20, 2017
@indirectlylit indirectlylit added P0 - critical Priority: Release blocker or regression and removed P1 - important Priority: High impact on UX labels Nov 20, 2017
@DXCanas DXCanas self-assigned this Nov 20, 2017
@radinamatic radinamatic removed the TODO: needs gherkin update Add to our manual integration tests label Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 - critical Priority: Release blocker or regression TAG: new feature New user-facing feature TAG: user strings Application text that gets translated
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants