Differential D17305

Add KDE Connect SMS plugin
ClosedPublic
Actions

Authored by nicolasfella on Dec 2 2018, 12:47 PM.

Details

Reviewers
apol
Commits
R495:617e09707f0f: Add KDE Connect SMS plugin
Summary

Enable sharing links via KDE Connect SMS

Depends on D17292
Depends on D17285

Test Plan

Send a page from Falkon via SMS

Diff Detail

Repository
R495 Purpose Library
Branch
kdeconnectsms
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 5870
Build 5888: arc lint + arc unit
nicolasfella created this revision.Dec 2 2018, 12:47 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 2 2018, 12:47 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
nicolasfella requested review of this revision.Dec 2 2018, 12:47 PM
apol added inline comments.Dec 4 2018, 2:41 AM
src/plugins/kdeconnect_sms/kdeconnectsmsplugin.cpp
51 ↗(On Diff #46697)

i18n? Thinking of RTL languages.

apol accepted this revision.Feb 8 2019, 10:36 PM
This revision is now accepted and ready to land.Feb 8 2019, 10:36 PM
Closed by commit R495:617e09707f0f: Add KDE Connect SMS plugin (authored by nicolasfella). · Explain WhyMar 9 2019, 10:06 PM
This revision was automatically updated to reflect the committed changes.

Showing Only Differences

This revision modifies 3 more files that are hidden because they were not modified between selected diffs and they have no inline comments.

Revision Contents
Changeset List

Diff 47226

View Options

src/plugins/kdeconnect_sms/CMakeLists.txt

Loading...
View Options

src/plugins/kdeconnect_sms/kdeconnectsmsplugin.cpp

Loading...