Skip to content

Add project layout, move scripts to /opt/zero2dash, update systemd units, and add framebuffer test script#2

Closed
aSbiEL0 wants to merge 4 commits intomainfrom
codex/create-placeholder-test-script
Closed

Add project layout, move scripts to /opt/zero2dash, update systemd units, and add framebuffer test script#2
aSbiEL0 wants to merge 4 commits intomainfrom
codex/create-placeholder-test-script

Conversation

@aSbiEL0
Copy link
Copy Markdown
Owner

@aSbiEL0 aSbiEL0 commented Feb 28, 2026

Motivation

  • Provide a sane on-device layout and deployable location for the Pi-hole TFT dashboard and simplify installation steps.
  • Add a lightweight local test utility to validate framebuffer rendering without the full Pi-hole integration.

Description

  • Added a suggested project structure to README.md and updated install steps to install python3-pil, copy files to /opt/zero2dash, and adjust service deployment commands to use /opt/zero2dash.
  • Moved existing scripts into a scripts/ directory and updated the WorkingDirectory and ExecStart/ExecStartPre paths in systemd/pihole-display.service and systemd/pihole-display-dark.service to point at /opt/zero2dash and /opt/zero2dash/scripts.
  • Added scripts/test_placeholder.py, a small utility that renders a centered "TEST" image, converts RGB888 to RGB565, and can either write to a framebuffer device or save a preview image via --output and skip framebuffer writes with --no-framebuffer.
  • Ensured the README documents the placeholder test command and the updated configuration path for piholestats_v1.2.py under /opt/zero2dash/scripts.

Testing

  • No automated tests were present or executed for these changes.

Codex Task

@aSbiEL0
Copy link
Copy Markdown
Owner Author

aSbiEL0 commented Feb 28, 2026

PS C:\zero2dash> git pull origin main
From https://github.com/aSbiEL0/zero2dash

  • branch main -> FETCH_HEAD
    Already up to date.
    PS C:\zero2dash> git checkout codex/create-placeholder-test-script
    branch 'codex/create-placeholder-test-script' set up to track 'origin/codex/create-placeholder-test-script'.
    Switched to a new branch 'codex/create-placeholder-test-script'
    PS C:\zero2dash> git merge main
    Committer identity unknown

*** Please tell me who you are.

Run

git config --global user.email "you@example.com"
git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'Default@Maciek-PC.(none)')
PS C:\zero2dash> git config --global user.email "asbiel0@gmail.com"
PS C:\zero2dash> git merge main
Auto-merging README.md
CONFLICT (content): Merge conflict in README.md
Auto-merging systemd/display.service
CONFLICT (content): Merge conflict in systemd/display.service
Automatic merge failed; fix conflicts and then commit the result.
PS C:\zero2dash>

@aSbiEL0
Copy link
Copy Markdown
Owner Author

aSbiEL0 commented Feb 28, 2026

@codex[agent] please fix the issues and create a new pull request

@Codex
Copy link
Copy Markdown
Contributor

Codex AI commented Feb 28, 2026

@aSbiEL0 I've opened a new pull request, #3, to work on those changes. Once the pull request is ready, I'll request review from you.

[WIP] Add project layout and update systemd units for Pi-hole TFT dashboard
@chatgpt-codex-connector
Copy link
Copy Markdown

Summary

  • Added compatibility systemd unit files systemd/display.service and systemd/display-dark.service so environments still referencing the legacy display*.service names can run with the new /opt/zero2dash and /opt/zero2dash/scripts paths. systemd/display.serviceL1-L21 systemd/display-dark.serviceL1-L21
  • Updated README.md to include the compatibility unit files in the project structure and to deploy all service units (systemd/*.service) during installation. README.mdL24-L31 README.mdL89-L96
  • Documented the compatibility intent explicitly in the README so users understand why both pihole-display*.service and display*.service exist. README.mdL138-L140
  • Committed the follow-up changes on branch work (67b8a33) and created a new PR via the make_pr tool with a title/body describing these fixes.

Testing

  • python3 -m py_compile scripts/test_placeholder.py scripts/piholestats_v1.1.py scripts/piholestats_v1.2.py
  • ⚠️ curl -s https://api.github.com/repos/aSbiEL0/zero2dash/contents/systemd | head -c 800 (used for upstream context check; returned 404 Not Found because the repository path is not publicly accessible from this environment)

View task →

@aSbiEL0 aSbiEL0 closed this Feb 28, 2026
@aSbiEL0 aSbiEL0 reopened this Feb 28, 2026
@aSbiEL0
Copy link
Copy Markdown
Owner Author

aSbiEL0 commented Feb 28, 2026

@codex[agent] recheck the code

@Codex
Copy link
Copy Markdown
Contributor

Codex AI commented Feb 28, 2026

@aSbiEL0 I've opened a new pull request, #7, to work on those changes. Once the pull request is ready, I'll request review from you.

Co-authored-by: openai-code-agent[bot] <242516109+Codex@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants