Skip to content

[DO NOT MERGE] rewrite work#82

Draft
ghost wants to merge 117 commits intolatestfrom
rt
Draft

[DO NOT MERGE] rewrite work#82
ghost wants to merge 117 commits intolatestfrom
rt

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jan 17, 2026

self explanatory

evil chipo and others added 30 commits December 20, 2025 06:14
- Lower Python version requirement from 3.14 to 3.13 for compatibility
- Remove missing QML import (pages/playerlog) from MainWindow.qml
- Generate translation QM files for English and Vietnamese
- Add missing config file and RinUI components
- Enable standalone executable compilation with all dependencies bundled
fix: make application buildable with PyInstaller
- Wellcome → Welcome (main greeting text)
- Unknown → Unknown (item status text)
- Intergation → Integration (menu item)
- Still working on this!! → Page under construction
- Open bloxstrap's document. → Open documentation.

Fixes affect all 31 translation files for improved professionalism
- Add build.py script that automatically generates resources and builds executable
- Update uv.lock with Python 3.13 compatibility for broader support
- Simplifies build process to single command: ./build.py
- build.sh handles environment setup, resource generation, and compilation
- Single command build process: ./build.sh
- Includes proper error handling and progress indicators
@ghost ghost marked this pull request as draft January 17, 2026 16:53
@ghost ghost added the working in progress this pr is not done yet label Jan 17, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2770025edc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/modules/backend.py
Comment on lines +104 to +106
os.path.expanduser(
f"{cfg.get_row("Sober", "Path")}/asset_overlay/content/fonts/"
),
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Fix invalid f-string quoting to avoid SyntaxError

The f-string in the Replace(...) call embeds cfg.get_row("Sober", "Path") inside a double-quoted f-string literal, which is a parse-time SyntaxError. Because Backend is imported at startup, this prevents the entire app from launching on any code path. Use different quote types or build the path separately before the f-string so the module can import.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

working in progress this pr is not done yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants