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

turn on bluetooth with rfkill #145

Closed
wants to merge 3 commits into from
Closed

turn on bluetooth with rfkill #145

wants to merge 3 commits into from

Conversation

torikulhabib
Copy link
Contributor

this branch dependen this elementary/wingpanel-indicator-bluetooth#129
Please comment again if something goes wrong.
in the branch that I commented on is probably understood.
because I am not fluent in English. there may be some delivery that is difficult to understand

@tintou
Copy link
Member

tintou commented Sep 24, 2020

Thanks for this patch, bu I don't think that we should go this route of doing everything manually in the indicator and plug.
The GNOME Settings Daemon already provides an interface on the session bus org.gnome.SettingsDaemon.Rfkill, we should start using it as it would allow us to have the state synced across all the components without much hassle.

@torikulhabib
Copy link
Contributor Author

previously I used the dbus method call org.gnome.SettingsDaemon.Rfkill then I kept trying for a few weeks, at one point I tried to turn on and off bluetooth org.gnome.SettingsDaemon.Rfkill didn't respond. and I tried to read the properties using d-feet and it doesn't change. then i unblock rfkill manually bluetooth it works. and org.gnome.SettingsDaemon.Rfkill still does not show changes until the new restart works.

@jeremypw jeremypw added the Status: Design Conflict Not going to be addressed due to conflicts with the HIG or other design considerations label Jun 29, 2021
@jeremypw
Copy link
Contributor

Closing as "Design Conflict" as the branch it depends on has already been closed as such. Thanks for your contribution anyway.

@jeremypw jeremypw closed this Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Design Conflict Not going to be addressed due to conflicts with the HIG or other design considerations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants