Skip to content

refactor: utilize appstream lib#151

Merged
jardon merged 11 commits intomainfrom
refactor/appstream-lib
Feb 1, 2026
Merged

refactor: utilize appstream lib#151
jardon merged 11 commits intomainfrom
refactor/appstream-lib

Conversation

@jardon
Copy link
Owner

@jardon jardon commented Jan 26, 2026

📝 Description

Please include a summary of the change and the issue it fixes.
Link to the related issue if applicable (e.g., Fixes #123).

Example:
This PR updates the login form validation logic to handle empty username fields.
Fixes #45.


🔍 Type of Change

Please delete options that are not relevant.

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code cleanup or refactor
  • 🧪 Tests
  • 📝 Documentation update
  • ⚙️ CI/CD or build system update

✅ How Has This Been Tested?

Describe the tests you ran to verify your changes.
Provide instructions so others can reproduce the results.

Example:

  • Unit tests added/updated
  • Manual testing on Chrome, Firefox, and Safari

Test Configuration:

  • OS:
  • Browser:
  • Node version (if applicable):

📸 Screenshots (if applicable)

Add screenshots or GIFs to help explain your changes visually.


📋 Checklist

Before submitting your PR, please check that:

  • Code follows the project’s style guidelines
  • I have self-reviewed my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation (if needed)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

🧠 Additional Context

Add any other context about the PR here (e.g., related PRs, implementation notes, trade-offs).

jardon added 11 commits January 26, 2026 07:44
- remove xml parser from flatpak backend
- rework getInstalledPackages to return list of app ids that are
  installed
- refactor Application class to line more closely with the appstream
  spec
- refactor Application test data to match Application class changes
- refactor app widgets to utilize the appstream lib
- watch featuredAppList on loading widget
- remove no longer needed helper classes: Screenshot, Release, Bundle
- replace hardcoded references to the default C
- add localized functions to application class
@jardon jardon marked this pull request as ready for review February 1, 2026 00:44
@jardon jardon merged commit 38ce8e8 into main Feb 1, 2026
6 checks passed
@jardon jardon deleted the refactor/appstream-lib branch February 1, 2026 00:48
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