Skip to content

PYL-39 POC non interactive user interface for pylms#27

Merged
lesaint merged 2 commits intomainfrom
sebastienlesaint/pyl-39-poc-non-interactive-user-interface-for-pylms
May 29, 2024
Merged

PYL-39 POC non interactive user interface for pylms#27
lesaint merged 2 commits intomainfrom
sebastienlesaint/pyl-39-poc-non-interactive-user-interface-for-pylms

Conversation

@lesaint
Copy link
Copy Markdown
Owner

@lesaint lesaint commented May 27, 2024

WHY

Practical use case for using tkinter after learning it last week

WHAT

Let's make the UI as similar to the command line as possible and composed of:

  1. one entry field at the top
    • the same string as the command line will be typed there, except for the name of the program (pylms)
  2. a text area below
    • it will display the same output as the command line

In other words:

  1. foo searches for a person matching foo and display the output in the text area
  2. hitting ENTER on an empty entry field will display all persons in the text area
  3. create donut creates a person with first name donut, clear the entry field, clear the text area and displays any log in the text area
    • create foo bar is also supported
  4. commands update, link and delete are not supported

HOW

  • Tkinter is used (obviously)
  • xvfb is used to run Tkinter test in Github actions
    • use /usr/bin/xvfb-run --auto-servernum make test to run tests locally with Xvfb and reproduce the errors specific to Xvfb

@lesaint lesaint changed the title PYL-39 poc non interactive user interface for pylms PYL-39 POC non interactive user interface for pylms May 27, 2024
@lesaint lesaint force-pushed the sebastienlesaint/pyl-39-poc-non-interactive-user-interface-for-pylms branch 3 times, most recently from a7aa6f9 to e9ab219 Compare May 29, 2024 19:33
@lesaint lesaint force-pushed the sebastienlesaint/pyl-39-poc-non-interactive-user-interface-for-pylms branch 2 times, most recently from 1d6780d to 2153964 Compare May 29, 2024 19:42
* use xvfb to run test in CI in a headless environement
@lesaint lesaint force-pushed the sebastienlesaint/pyl-39-poc-non-interactive-user-interface-for-pylms branch from 2153964 to 55c8408 Compare May 29, 2024 19:45
@sonarqubecloud
Copy link
Copy Markdown

@lesaint lesaint merged commit a508a0a into main May 29, 2024
@lesaint lesaint deleted the sebastienlesaint/pyl-39-poc-non-interactive-user-interface-for-pylms branch May 29, 2024 19:56
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