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

Add API for setting host apps AppID #257

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bilelmoussaoui
Copy link
Owner

No description provided.

src/desktop/camera.rs Outdated Show resolved Hide resolved
src/registry.rs Outdated Show resolved Hide resolved
/// verifies that the application is not sandboxed. It then uses the
/// `org.freedesktop.host.portal.Registry` interface to register the process
/// communicating over DBus with the portal as the owner of the specified
/// application ID. For more technical details, see ...
Copy link
Collaborator

@A6GibKm A6GibKm Jan 14, 2025

Choose a reason for hiding this comment

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

On thing I am missing is when to call this, e.g. we had problems with the camera portal if it was called too early. cc @GeorgesStavracas

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also the ... should be replaced before merging.

@A6GibKm
Copy link
Collaborator

A6GibKm commented Jan 14, 2025

For anyone wondering, this depends on flatpak/xdg-desktop-portal#1521

@bilelmoussaoui bilelmoussaoui force-pushed the bilelmoussaoui/registry branch from 51ddc94 to 65f5d6b Compare January 15, 2025 01:57
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.

3 participants