Skip to content

feat(dbus): allow everyone to use cardwire's API - #253

Merged
luytan merged 1 commit into
mainfrom
dbus-allow-all
Sep 20, 2026
Merged

luytan merged 1 commit into
mainfrom
dbus-allow-all

Conversation

@luytan

@luytan luytan commented Sep 20, 2026

Copy link
Copy Markdown
Member

Description

Allow everyone to use cardwire's dbus API

Fixes #242 and bazzite ublue-os/bazzite#5866

Checklist:

  • My code follows the style guidelines of this project (cargo fmt)
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the mdBook documentation
  • My changes generate no new warnings (clippy/clang)
  • New and existing unit tests pass locally with my changes (either use nix flake check or wait for the ci)

@coderabbitai

coderabbitai Bot commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 6ceedd25-68f6-4c99-b137-551e3994a01a

📥 Commits

Reviewing files that changed from the base of the PR and between e9aaa50 and 1948eed.

📒 Files selected for processing (1)
  • assets/org.opengamingcollective.cardwire.conf
 _________________________________________________________________________________
< CodeRabbi is my uncle. He's a bit more orthodox. Feel lucky you got me instead. >
 ---------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@luytan
luytan merged commit 2584ff6 into main Sep 20, 2026
7 of 8 checks passed
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.

Feat: allow non sudo/wheel users to use dbus

1 participant