-
Notifications
You must be signed in to change notification settings - Fork 168
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
Shortcut: Append to note #1170
Shortcut: Append to note #1170
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@charliescheer sending you a few notes!! only important one is related to the entitlements.
TY!!
IntentsExtension/Support Files/IntentsExtensionDebug.entitlements
Outdated
Show resolved
Hide resolved
@@ -10,4 +10,7 @@ import Foundation | |||
|
|||
struct IntentsConstants { | |||
static let noteIdentifierKey = "OpenNoteIntentHandlerIdentifierKey" | |||
static let extensionGroupName = Bundle.main.sharedGroupDomain | |||
static let simperiumAppID = "chalk-bump-f49" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably come from SimplenoteConstants.simperiumAppID
Please make sure that file doesn't get uploaded!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Generated by 🚫 Danger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix
One of the most likely and useful functions we could add to Simplenote shortcuts is the ability to append to a note. So in this PR I have done so.
You can setup a shortcut to append a specific content to any note, or you can get the note from another shortcut intent.
The append intent uses the simperium api so the server can handle any diffing or sync that needs to happen and the main app does not need to be open for the shortcut to work and then sync.
relies on #1169
Test
Review
(Required) Add instructions for reviewers. For example:
Release