-
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: Open Note #1163
Shortcuts: Open Note #1163
Conversation
…5/open-note-intent-mk4
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.
✅ Verified that the Open Note shortcut effectively opens the target note
Sending you a couple questions!! TY!!
@@ -0,0 +1,61 @@ | |||
// This file contains the required class structure to be able to fetch and use core data files in widgets and intents |
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.
Nipticky: Filename should probably be Note+Intents
?
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 is so cool! Worked well for me.
What do you think about maybe focusing the note editor after opening the note? So say I have a "Shopping List" note, it'd be cool if it opened with the editor active so I could type "beer" "soda" straight away.
Good idea, TY. Done! |
Fix
Okay, so back on track after a bunch of setup PRs. What we have here is the ability to open a note from a shortcut. That should be fairly simple, but there were a lot of core technologies that needed to be added to do this. So sorry but this PR is a little long. One the plus side several hundred lines of it are all xml files that are auto generated and shouldn't need much review.
Anyways, so the shortcut that I have added here allows you to specify or search for a note and then open it in Simplenote. Straight forward, but it is defining some of the core tech we will need for more advanced shortcuts.
Moves forward: #1145
relies on #1162
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: