-
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
Shortcuts: Add open new note intent #1160
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.
Added a note on the ShortcutHandler
component.
Other than that, feel free to when ready!
@@ -8,8 +8,12 @@ | |||
<dict> | |||
<key>IntentsRestrictedWhileLocked</key> | |||
<array/> | |||
<key>IntentsSupported</key> | |||
<key>IntentsRestrictedWhileProtectedDataUnavailable</key> |
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.
TIL!
This is GTG, just waiting on the PR that this relies on |
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.
|
||
switch type { | ||
case .newNoteShortcut: | ||
SimplenoteAppDelegate.shared().noteEditorViewController.createNote(from: nil) |
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.
You can drop SimplenoteAppDelegate.shared()
(because that should be self
)
Fix
So the very first shortcut to add is one that simply opens Simplenote and creates a new note. This is a basic proof of concept that the intents and shortcut handling works.
So in this PR you can create a shortcut "Open New Note" that will launch Simplenote and then create a new note.
Relies on #1159
Test
Review
(Required) Add instructions for reviewers. For example:
Release
(Required) Add a concise statement to
RELEASE-NOTES.txt
if the changes should be included in release notes. Include details about updating the notes in this section. For example: