Skip to content
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

Merged
merged 17 commits into from
May 31, 2024
Merged

Shortcuts: Open Note #1163

merged 17 commits into from
May 31, 2024

Conversation

charliescheer
Copy link
Contributor

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

  1. build and launch Simplenote. Make sure you are logged in
  2. Open the shortcuts app, create a new shortcut, find simplenote, select Open Note
  3. In the shortcut that gets created select a note to open
  4. Run the shortcut. Simplenote should launch and open that note.

Review

(Required) Add instructions for reviewers. For example:

Only one developer and one designer are required to review these changes, but anyone can perform the review.

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:

These changes do not require release notes.

@charliescheer charliescheer added the [feature] shortcuts Anything related to Shortcuts label May 29, 2024
@charliescheer charliescheer added this to the 2.21 milestone May 29, 2024
@charliescheer charliescheer self-assigned this May 29, 2024
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

Copy link
Contributor

@jleandroperez jleandroperez left a 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
Copy link
Contributor

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 ?

IntentsExtension/Models/SPManagedObject+Widget.swift Outdated Show resolved Hide resolved
Simplenote/SimplenoteAppDelegate+Swift.swift Outdated Show resolved Hide resolved
Base automatically changed from charlie/migrate-database-to-group-container to trunk May 31, 2024 17:28
Copy link
Contributor

@roundhill roundhill left a 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.

@charliescheer
Copy link
Contributor Author

Good idea, TY. Done!

@charliescheer charliescheer merged commit 6c01409 into trunk May 31, 2024
8 checks passed
@charliescheer charliescheer deleted the charlie/1145/open-note-intent-mk4 branch May 31, 2024 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[feature] shortcuts Anything related to Shortcuts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants