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 update part 4: reimplement open note with siriintents #1581

Merged

Conversation

charliescheer
Copy link
Contributor

Fix

Next step in implementing shortcuts with siri kit is to implement the open note shortcut. This allows you to open into Simplenote in any previously selected note.

fixes #1568

Test

  1. Run the branch, go to the shortcuts app, create a new shortcut, add action, find Simplenote and select the open note shortcut.
  2. Pick a note to open
  3. run the shortcut and confirm that Simplenote opens that note

Review

(Required) Add instructions for reviewers. For example:

Only one developer is required to review these changes, but anyone can perform the review.

Release

These changes do not require release notes.

@charliescheer charliescheer added this to the 4.52 milestone May 3, 2024
@charliescheer charliescheer self-assigned this May 3, 2024
@charliescheer charliescheer changed the title Charlie/1568/reimplement open note with siriintents Shortcuts update part 4: reimplement open note with siriintents May 3, 2024
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 the new Open Note Intent effectively opens the target note

Nice work!!

:shipit:

@@ -32,7 +32,8 @@
<array>
<string>ListWidgetIntent</string>
<string>NoteWidgetIntent</string>
<string>SPOpenNewNoteIntent</string>
<string>OpenNewNoteIntent</string>
<string>OpenNoteIntent</string>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noooow we're talking!! 😎

@wpmobilebot
Copy link
Collaborator

You can test the changes in simplenote-ios from this Pull Request by:

  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr1581-7655a6d-018f53e5-11fe-4f71-bf90-f135c743c566 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

Base automatically changed from charlie/1568/reimplement-open-note-in-siri-intents to trunk May 7, 2024 16:59
@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

@charliescheer charliescheer merged commit d656557 into trunk May 7, 2024
11 checks passed
@charliescheer charliescheer deleted the charlie/1568/reimplement-open-note-with-siriintents branch May 7, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reimplement existing shortcuts in App Intents
4 participants