search

b4j4 / indicator-kdeconnect

AppIndicator for KDE Connect
GNU Lesser General Public License v2.1
371 stars 38 forks source link

Uses the Dbus Interface to send sms #97

Closed Marukesu closed 7 years ago

Marukesu commented 7 years ago

the actual method fail if the user uses the kdeconnect flatpak because the diferent nomeclature and the flatpak run --comand="kdeconnect-cli" org.kde.kdeconnect don't run any variable, only show the help message. so using the dbus interface directly allows send sms from both flatpak and non-sandboxed kdeconnect.

b4j4 commented 7 years ago

@MrMarukesu the Sms.py is executed by terminal, that is a problem or just the act of send SMS?

Marukesu commented 7 years ago

just sending the sms, the sms.py run correctly

andyholmes commented 7 years ago

Submitted PR #98

b4j4 commented 7 years ago

@MrMarukesu check the merge from #98 and give any feedback.

Marukesu commented 7 years ago

sms works now, thanks.

b4j4 commented 7 years ago

@MrMarukesu so when it's finished, feel free to make a pull request with your work.