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

Hack Week experiments #1

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Hack Week experiments #1

wants to merge 2 commits into from

Conversation

ancorgs
Copy link
Owner

@ancorgs ancorgs commented Nov 10, 2023

This is not really a pull request since I don't have any intention to merge this.

This is the result of https://hackweek.opensuse.org/23/projects/tui-interface-for-agama

The main goal was learning Rust because I had zero experience with it. As a consequence of that lack of experience the result is not exactly impressive.

Despite the EXTREMELY low quality of the code, this tool already does some basic stuff:

  • Connects to Agama to get the list of available devices and the one that is selected for installation
  • Initializes the corresponding selector in the UI
  • When the selector in the UI changes, it updates Agama configuration

Many of the things I wanted to do were left out due to lack of time (it took me almost the whole week to start being minimally productive with Rust and tui-realm):

  • Subscribing to Agama D-Bus interface to dynamically update the UI on changes
  • Initializing and updating the list of actions (which is in fact a particular case of the former point).

tui-realm-hw

joseivanlopez pushed a commit that referenced this pull request Aug 21, 2024
## Problem

[Switching from the former UI to the current
approach](agama-project#1202) left behind a lot
of things to review, dead components and a CSS clean up among them.

## Solution

Drop dead components and make as much clean-up rounds as needed. This
can be considered `Round #1`, made in the middle of more urgent work.

## Testing

Tested manually

## Screenshots

Visual changes after this PR should keep at minimum. In any case, it's
preferred that reviewers give the UI a shot taking the time they need.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant