-
-
Notifications
You must be signed in to change notification settings - Fork 326
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
Implement JSONRPC V2 #1353
Implement JSONRPC V2 #1353
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
# Conflicts: # Flow.Launcher/Flow.Launcher.csproj # Flow.Launcher/Notification.cs
This comment has been minimized.
This comment has been minimized.
# Conflicts: # Flow.Launcher.Plugin/Query.cs
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
what do we need to do to get current plugins to start using v2 ? |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@check-spelling-bot Report🔴 Please reviewSee the 📂 files view, the 📜action log, or 📝 job summary for details. Unrecognized words (23)
To accept these unrecognized words as correct, you could run the following commands... in a clone of the git@github.com:Flow-Launcher/Flow.Launcher.git repository curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/v0.0.22/apply.pl' |
perl - 'https://github.com/Flow-Launcher/Flow.Launcher/actions/runs/6752180862/attempts/1' OR To have the bot accept them for you, reply quoting the following line: Pattern suggestions ✂️ (1)You could add these patterns to
Errors (5)See the 📂 files view, the 📜action log, or 📝 job summary for details.
See ❌ Event descriptions for more information. If the flagged items are 🤯 false positivesIf items relate to a ...
|
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.
Good work thank you for implementing V2.
Before we can publicly announce this new feature, we will need to:
- Add documentation
- Add the example plugin you attached in this PR as HelloWorld plugin
- Add a way to automatically transition/convert existing plugins to V2
Let's make sure people can create V2 plugins easily.
Tasks:
Query
object to plugin as argumentFinish Python Script example withijson
Implement API call (we need a new identifier)Motivation:
Currently our jsonrpc plugins are very limited. They are not able to visit any api that might receive a result in Flow. They are not able to cache in memory as flow run every single query in different instance.
This PR:
F5
so the development is no harder than beforeSample Python Code
jsonrpcv2.zip
Limitation
TypeError
is used to detect whether flow sends the correct argument to the server, but this errors can also be raised inside the method.Future Work