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

Move system administration apps to advanced section #36

Merged
merged 6 commits into from
Aug 10, 2023

Conversation

ethanjli
Copy link
Member

This PR moves the links for Cockpit, Portainer, and the system file browser from the main "Browser applications" section into a "Browser applications" subsection under the "For advanced users" section. This change was requested by Satoshi in https://planktoscope.slack.com/archives/C01V5ENKG0M/p1688404265915379?thread_ts=1686544549.181279&cid=C01V5ENKG0M because of the risk of the user accidentally making changes which break their system. In particular, Satoshi broke the PlanktoScope's network configuration by touching Cockpit's network page (which attempts to set up systemd-networkd over our dhcpcd-based network configuration). We will definitely need to provide documentation about how to use these browser applications safely (perhaps linking to the relevant documentation pages from the applications' descriptions on the device-portal home page) before we can consider moving them into the general-usage "Browser applications" section of the device-portal home page.

This PR also moves the link for the Node-RED software editor into the "For advanced users" section for the same reason - because the user can unintentionally cause breakage in the PlanktoScope.

@ethanjli ethanjli enabled auto-merge August 10, 2023 22:19
@ethanjli ethanjli disabled auto-merge August 10, 2023 22:36
@ethanjli ethanjli merged commit 1ae0ce1 into main Aug 10, 2023
@ethanjli ethanjli deleted the feature/move-admin-apps branch August 10, 2023 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

1 participant