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

Extended filtering #15

Merged
merged 5 commits into from Feb 15, 2022
Merged

Conversation

myxor
Copy link
Contributor

@myxor myxor commented Feb 12, 2022

Possible filter types:

  • .../BangleApps/#blue shows apps having "blue" in app.id or app.tag --> searchType:hash (as before these changes)
  • .../BangleApps/#bluetooth shows apps having "bluetooth" in app.id or app.tag (also selects bluetooth chip) --> searchType:chip (as before these changes)
  • .../BangleApps/id=antonclk shows app having app.id == antonclk --> searchType:id
  • .../BangleApps/q=clock shows apps having "clock" in app.id or app.description --> searchType:full

Typing in the search field on the web interface does full search now (and the URL is changed according to the input value).

Fixes espruino/BangleApps#1215

Note: there is no change in the sorting as suggested in comment espruino/BangleApps#1215 (comment) done.

@gfwilliams even though i tested this i would like you to have a second look on this before merging :) Thanks!

@myxor myxor marked this pull request as draft February 12, 2022 23:12
@myxor myxor changed the title WiP: Extended filtering Extended filtering Feb 14, 2022
@myxor myxor marked this pull request as ready for review February 14, 2022 14:52
@gfwilliams
Copy link
Member

Looks good to me! Not sure if you have any idea but it seems like some kind of small 'copy link' button might be neat on apps so that it's obvious how to share them.

On the forum I find I often want to say 'look at the readme for X' - so I wonder if there's a way to do that...

@gfwilliams gfwilliams merged commit bd894bf into espruino:master Feb 15, 2022
@myxor
Copy link
Contributor Author

myxor commented Feb 15, 2022

Not sure if you have any idea but it seems like some kind of small 'copy link' button might be neat on apps so that it's obvious how to share them.

I will have a look at this - should not be that hard.

@gfwilliams
Copy link
Member

Thanks - that'd be great!

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.

AppLoader: Search on description
2 participants