Skip to content

Set app id via registry portal - #8988

Merged
sithlord48 merged 4 commits into
masterfrom
setAppIdViaRegistryPortal
Sep 30, 2025
Merged

Set app id via registry portal#8988
sithlord48 merged 4 commits into
masterfrom
setAppIdViaRegistryPortal

Conversation

@sithlord48

@sithlord48 sithlord48 commented Sep 29, 2025

Copy link
Copy Markdown
Member

Related #8869

It turns out we did not quite finish the restore part when running from outside a sandboxed deployment (i.e flatpak)

  • Detect when in a sandbox either flatpak or snap
  • Use the xdg-registry-portal to set the app id. when not in a sandbox
  • set new variable in cmake CMAKE_PROJECT_FQDN_NAME to use for the fully reverse domain name (i.e org.deskflow.deskflow) set it on constants.h use it where were used to use the string directly.

@sithlord48 sithlord48 added this to the v1.25.0 milestone Sep 29, 2025
@sithlord48
sithlord48 requested a review from nbolton September 29, 2025 22:46
@sithlord48 sithlord48 added the wayland This issues effects Wayland on Linux label Sep 29, 2025
@sithlord48
sithlord48 force-pushed the setAppIdViaRegistryPortal branch 4 times, most recently from 8692796 to 83fd1c0 Compare September 30, 2025 01:48
@sithlord48
sithlord48 force-pushed the setAppIdViaRegistryPortal branch from 83fd1c0 to 52594a2 Compare September 30, 2025 02:08

@nbolton nbolton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, nice refactor too 👍

OOO, can't test.

@sithlord48
sithlord48 merged commit 6f574a3 into master Sep 30, 2025
31 checks passed
@sithlord48
sithlord48 deleted the setAppIdViaRegistryPortal branch September 30, 2025 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wayland This issues effects Wayland on Linux

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants