Skip to content

Add a basic search page#25

Draft
radostin04 wants to merge 2 commits into
pebble-dev:mainfrom
radostin04:issue-2
Draft

Add a basic search page#25
radostin04 wants to merge 2 commits into
pebble-dev:mainfrom
radostin04:issue-2

Conversation

@radostin04
Copy link
Copy Markdown

This implements #2

I've added a basic search page that fetches and displays search results from Algolia. I've also implemented the beginnings of a horziontal layout for AppCard, intended to emulate the horizontal watchapps result page from the legacy appstore.

I would need more information to solve the following problems:

  • Screenshots do not currently display, as the screenshot format returns in the app info objects by Algolia is different from the one that AppCard expects. I assume this is intentional and that I should implement handling regardless, but I'd like to confirm beforehand
  • Watchapps do not provide a list_image, only screenshots - the legacy appstore also only displayed screenshots for watchapps, but I think this is a bit weird - if the main appstore page displays icons, so should the search. Of course, to make this possible, list_image would need to be added to the Algolia search results - otherwise, I can replicate the legacy behavior of showing a screenshot
  • I've hardcoded the URL and API key at this time - Is there a centralized place where URLs and configurations and such are stored? I couldn't find one.
  • Pagination isn't implemented at this time - how many results should there be per page? The legacy appstore appears to use 39 for watchfaces and 15 for apps.

Please @ me on Discord if you reply to this, as I for the life of me cannot get GitHub to send me notifications.

Adds a basic search page that fetches and displays results from Algolia.
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.

1 participant