-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Provide a Flatpak manifest #34
Conversation
I could use some help. Feedback starts now, but the submenus for Default Apps, Panel Indicators and System Settings are empty. I assume that they are filled dynamically and that starting it as Flatpak will not find the applications. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need some cleanups definitions and others minors changes.
about the appstream error, the flatpak don't have access to the host metadata, so we need to include the needed files with the flatpak, or place them in a location that the flatpak can read if setted with it fix the Painel Indicators and System Settings, but for Default Appliications it fails in create the |
Co-authored-by: Gustavo Marques <[email protected]>
Co-authored-by: Gustavo Marques <[email protected]>
Co-authored-by: Gustavo Marques <[email protected]>
Co-authored-by: Gustavo Marques <[email protected]>
Co-authored-by: Gustavo Marques <[email protected]>
Co-authored-by: Gustavo Marques <[email protected]>
@Marukesu thanks for your feedback! @danrabbit what is your opinion how we want to continue? |
I think it would definitely be preferable to poke the right holes in the sandbox so it can read the installed apps from the system. It would be ultimately more maintainable to reflect what the user actually has installed |
adding
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This at least builds and shows apps, I think we can follow up later with a solution for all app icons. That probably means adding icons to other components appdata files. So I think this is good to merge :)
Closes #33