-
Notifications
You must be signed in to change notification settings - Fork 7
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
Search Saves via Siri or App Shortcuts #1060
Conversation
Analytics: Coverage: 59.14
Pocket.app: Coverage: 75.56
PocketKit: Coverage: 62.61
AnalyticsTests: Coverage: 24.08
PocketKitTests: Coverage: 28.98
Generated by 🚫 Danger Swift against 2001381 |
3cd9a62
to
03aaf0a
Compare
47a2797
to
eb96681
Compare
8dd7e90
to
9c6d7bf
Compare
e8104fe
to
4c7b946
Compare
@@ -0,0 +1,3 @@ | |||
"Show me some cool stuff in ${applicationName}" = "Zeig mir coole Sachen in ${applicationName}"; |
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.
Should we setup this file path in smartling?
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.
eventually, though I am still trying to figure out if we can leverage our Localization package and SwiftGen, so the localization on Intents is still kinda WIP, but this works nonetheless.
b1b99d9
to
2001381
Compare
…rch bar is the first responder when search is activated via intent
…o ensure the searchController life cycle is triggered correctly
2001381
to
85702ed
Compare
Goal
Note
As of now, the search is intentionally limited to
Saves
(notArchive
norAll
)Test Steps
Issue(s)
POCKET-10508
Video
Screen.Recording.2024-08-29.at.4.35.53.PM.mov