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

Drop org.kde.* own-name #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

bbhtt
Copy link

@bbhtt bbhtt commented Sep 28, 2023

This is no longer required with any supported runtimes, the issue was fixed in Qt

https://docs.flatpak.org/en/latest/desktop-integration.html#statusnotifier

Most implementations of StatusNotifer have dropped this requirement

https://github.com/flathub/flatpak-builder-lint/issues/ 66#issuecomment-1386033025

I assume this was for tray, if it was for anything else, it needs to be figured out.

This is no longer required with any supported runtimes, the issue was fixed in Qt

https://docs.flatpak.org/en/latest/desktop-integration.html#statusnotifier

> Most implementations of StatusNotifer have dropped this requirement

https://github.com/flathub/flatpak-builder-lint/issues/ 66#issuecomment-1386033025
@flathubbot
Copy link
Contributor

Started test build 69366

@flathubbot
Copy link
Contributor

Build 69366 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/52020/org.easycoding.TunedSwitcher.flatpakref

@bbhtt
Copy link
Author

bbhtt commented Sep 28, 2023

Needs testing because I can't get the app to run. It closes immediately after entering password with "could not open network socket" in a KDE neon VM.

@bbhtt bbhtt requested a review from xvitaly January 3, 2024 04:41
@bbhtt
Copy link
Author

bbhtt commented Jan 3, 2024

This will now be blocked through the linter https://docs.flathub.org/docs/for-app-authors/linter. Can you please confirm that this works?

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.

2 participants