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

Add Universal Search Suggestions #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Fefedu973
Copy link

@Fefedu973 Fefedu973 commented Jun 22, 2024

[Insert URL to the plugin here]

[Adds search suggestions when typing something.]

By submitting this pull request I confirm I've read and complied with the below requirements 🖖

Requirements for your pull request

  • You have read and understood the Contribution Guidelines.
  • This pull request has a title in the format Add [Name of Plugin].
    • Add GEmojiSharp
    • Update readme.md
    • Adding GEmojiSharp
    • Added GEmojiSharp
  • Your entry should include a short description of the plugin. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb.
    • - [GEmojiSharp](…) - Find and copy GitHub Emoji to the clipboard.
    • - [GEmojiSharp](…) - A GitHub Emoji plugin for PowerToys Run.
    • - [GEmojiSharp](…)
  • The link text should be the Name of the plugin found in the plugin.json file. The URL should be the link to the GitHub repo, or a link to an anchor in the readme.
    • - [GEmojiSharp](https://github.com/hlaueriksson/GEmojiSharp) - …
    • - [GEmojiSharp.PowerToysRun](…) - …
    • - [GEmojiSharp](https://github.com/hlaueriksson/GEmojiSharp/releases/download/v1.0.0/GEmojiSharp.PowerToysRun.1.0.0.zip) - …
  • Your entry should be added at the bottom of the appropriate category.

@Fefedu973
Copy link
Author

@hlaueriksson

@hlaueriksson
Copy link
Owner

Hi @Fefedu973

Nice plugin!

It's similar to my Bang plugin.

I created an issue in your repo with feedback.

@Fefedu973
Copy link
Author

Fefedu973 commented Jul 7, 2024

Hi @Fefedu973

Nice plugin!

It's similar to my Bang plugin.

I created an issue in your repo with feedback.

Yeah I saw it, thanks !
The first time I saw your plugin was after I finished mine (I was a little sad to have done something that already existed lol) but they are not 100% the same so it is still nice.
It would be nice if you put your plugins on this page

@hlaueriksson
Copy link
Owner

Yeah, I should put my plugins here. Sometimes it feels awkward to toot my own horn.

@Fefedu973
Copy link
Author

Yeah, I should put my plugins here. Sometimes it feels awkward to toot my own horn.

Yeah I understand (this what i though when i saw that you didn't put your own plugins on this page) but i think you deserve it and that thoses plugins deserve a place here

@Fefedu973
Copy link
Author

Yeah, I should put my plugins here. Sometimes it feels awkward to toot my own horn.

The feedback has been taken into account !

@hlaueriksson
Copy link
Owner

Great work!

Can you update the PR title, name and description of the plugin?

@Fefedu973
Copy link
Author

yeah !

@Fefedu973 Fefedu973 changed the title Add Google Search Suggestions Add Universal Search Suggestions Jul 7, 2024
@Fefedu973
Copy link
Author

Done @hlaueriksson !

@Fefedu973
Copy link
Author

Is there anything else I need to change or is this all good?

Copy link

@Gtmz3 Gtmz3 left a comment

Choose a reason for hiding this comment

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

Nice

@hlaueriksson
Copy link
Owner

@Fefedu973 thanks for your contribution. However I'm not comfortable merging this until Fefedu973/PowerToys-Run-Plugin-Updater#1 is fixed.

This list is curated and built on trust. I cannot endorse a plugin that fails to satisfy the Security Policy.

@Fefedu973
Copy link
Author

Fefedu973 commented Jul 18, 2024

@Fefedu973 thanks for your contribution. However I'm not comfortable merging this until Fefedu973/PowerToys-Run-Plugin-Updater#1 is fixed.

This list is curated and built on trust. I cannot endorse a plugin that fails to satisfy the Security Policy.

How can I fix that ? Should I sign my software by buying the expensive Microsoft thing ?
Should I setup my GitHub repo to build the software directly in GitHub to be sure that the given exe is safe (directly from the GitHub code) or should I change the main behavior of my code?
Idk

@hlaueriksson
Copy link
Owner

The idea of updating the plugin from the PowerToy Run context itself is awesome.

Instead of downloading an external winui app, is it possible to do the same thing from PowerToy Run?

image

The code could be distributed via a NuGet package and used by plugins with the DynamicLoading setting in plugin.json.

I'm wondering how antivirus providers will consider this approach?

@Fefedu973
Copy link
Author

Oh no this means that I will need to throw my app away ): nevermind however the implementation of an updater directly in the plugin looks hard to me because while updating the plugin replaces its own files but the plugin needs to be running while doing the update process...

1 similar comment
@Fefedu973
Copy link
Author

Oh no this means that I will need to throw my app away ): nevermind however the implementation of an updater directly in the plugin looks hard to me because while updating the plugin replaces its own files but the plugin needs to be running while doing the update process...

@Fefedu973
Copy link
Author

But I think powertoys should add themselves an plugin updater in their side because on the plugin json we are providing the GitHub url so they can check the updates like this

@hlaueriksson
Copy link
Owner

Hi @Fefedu973

I created a NuGet package that adds support for updating PowerToys Run Plugins:

@Fefedu973
Copy link
Author

Very nice! I'm going to replace my updater software with yours, (I'm a little bit sad because I put a lot of Elbow grease in this project but yours is better and pass the antivirus test)
Thanks!

@hlaueriksson
Copy link
Owner

I'm sorry if my feedback came across as harsh.

Code that you throw away is never wasted. It serves as a learning experience. 🌱

@Fefedu973
Copy link
Author

I'm sorry if my feedback came across as harsh.

Code that you throw away is never wasted. It serves as a learning experience. 🌱

No this is not the feedback I fully understand but it is the code that I left behind, but you are right.
I'm currently trying to understand how to implement your package in my code but I wasn't able to understand the given code snippet in the readme, could you please explain to me briefly what are the different methods of the package and how to implement it In my case (I didn't understand for example why you set the plugin settings like this) I think that the given code lacks comments.
Thanks 👍

@hlaueriksson
Copy link
Owner

Take a look at the code here for now; https://github.com/hlaueriksson/GEmojiSharp/tree/master/src/GEmojiSharp.PowerToysRun

I will add better documentation in the readme tomorrow. 😇

@Fefedu973
Copy link
Author

I have already tried but the code is almost the same. I'm going to wait for the doc then
Thanks

@hlaueriksson
Copy link
Owner

I have updated the docs.

@Fefedu973
Copy link
Author

Thanks , I'll take a look

@hlaueriksson
Copy link
Owner

There are a few bugs I will try to fix in a 0.2.0 release.

@Fefedu973
Copy link
Author

Okay nice

@Fefedu973
Copy link
Author

the documentation is nice but the issue is with the main.cs code i really don't know why you manage settings like that or your code style in general. Is there anyway you give us just what we need to add to an aleready build project rather than a whole main.cs template where we are lost between what we should add to our code to make it work, what is aleready here but not exactly the same due to different coding styles. I think you get the point i'm lost lol 😭

@hlaueriksson
Copy link
Owner

hlaueriksson commented Aug 7, 2024

Take a look at the sample project:

I created that with the dotnet new template:

And then I added Community.PowerToys.Run.Plugin.Update to it.

So, if you create your own project with:

dotnet new ptrun-proj -o Community.PowerToys.Run.Plugin.Sample

You can diff the two projects and figure out what changes are needed to support updates.

Like so:

image

image

@Fefedu973
Copy link
Author

Oh, I didn't use that template for my project so this is why the coding style is different And I can't get out of it

1 similar comment
@Fefedu973
Copy link
Author

Oh, I didn't use that template for my project so this is why the coding style is different And I can't get out of it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants