Skip to content

Conversation

@ngtr6788
Copy link

Fixes #2049.

Hello everyone! This is my first time contributing to Flow Launcher, and I am very new at C# in general. My goal is (hopefully) get all the latest plugins into the store and show all the most up-to-date installed plugins, and allow the search function to work even after the refresh button is clicked. Can anyone check that the implementation is correct, and if not, let me know if there are any fixes?

Thanks so much!

@VictoriousRaptor VictoriousRaptor added the bug Something isn't working label May 14, 2023
@VictoriousRaptor VictoriousRaptor added this to the 1.15.1 milestone May 14, 2023
@VictoriousRaptor VictoriousRaptor requested a review from jjw24 May 14, 2023 13:40
@ngtr6788 ngtr6788 marked this pull request as draft May 15, 2023 04:37
@taooceros
Copy link
Member

Thank you so much for the fix! Would you please check whether it is possible to fix this bug while keeping using Command instead of a method callback? It doesn't seem to matter here but it might be better to keep the code consistent.

@ngtr6788 ngtr6788 force-pushed the search-plugin-store-refresh branch from 9cb01c3 to c5a4e4f Compare May 18, 2023 00:24
@ngtr6788 ngtr6788 force-pushed the search-plugin-store-refresh branch from c5a4e4f to eaaf7c2 Compare May 18, 2023 00:26
@ngtr6788 ngtr6788 marked this pull request as ready for review May 18, 2023 00:28
@ngtr6788
Copy link
Author

Thank you so much for the fix! Would you please check whether it is possible to fix this bug while keeping using Command instead of a method callback? It doesn't seem to matter here but it might be better to keep the code consistent.

Well, turns out it is possible. I just updated the PR :). You can take a look.

@VictoriousRaptor VictoriousRaptor merged commit c55edca into Flow-Launcher:dev May 18, 2023
@jjw24 jjw24 modified the milestones: 1.15.1, 1.16.0 Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: Plugin Store can not be searched after clicking refresh button

4 participants