Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Providers links from Source page not working properly #1008

Closed
1 task
krysal opened this issue Feb 25, 2022 · 4 comments · Fixed by #1166
Closed
1 task

Providers links from Source page not working properly #1008

krysal opened this issue Feb 25, 2022 · 4 comments · Fixed by #1166
Assignees
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟥 priority: critical Must be addressed ASAP

Comments

@krysal
Copy link
Member

krysal commented Feb 25, 2022

Description

The links from the Source column in the table of the Sources page return empty results in the search page. The issue here is that this is going to the general search (all media) which doesn't support the source filter and there is also no search term, this table was created when images were the only media available.

Reproduction

  1. Go to https://wordpress.org/openverse/sources or https://search-staging.openverse.engineering/sources and click in any source
  2. See the empty search result page.

Expectation

The links should lead to the image (or audio when added those sources to the table) search filtered by the clicked source, and therefore display a non-empty results page.

Resolution

  • 🙋 I would be interested in resolving this bug.
@krysal krysal added 🟧 priority: high Stalls work on the project or its dependents 🛠 goal: fix Bug fix 💻 aspect: code Concerns the software code in the repository labels Feb 25, 2022
@sarayourfriend
Copy link
Contributor

The links should lead to the image (or audio when added those sources to the table) search filtered by the clicked source, and therefore display a non-empty results page.

What should we do with providers like Wikimedia? 🤔

@krysal
Copy link
Member Author

krysal commented Feb 25, 2022

Good question, probably have to add it twice. I guess we need an extra column in the table to indicate the media type of the providers.

@sarayourfriend
Copy link
Contributor

sarayourfriend commented Feb 25, 2022

The table data is based off the ContentProvider's API right?

https://github.com/WordPress/openverse-api/blob/a7955c86d43bff504e8d41454f68717d79dd3a44/api/catalog/api/models/models.py#L29-L32

It doesn't look like it allows for "mixed" provider types, would we just add Wikimedia there twice?

Update: Oh dear, I should have just read the doc string 😅

https://github.com/WordPress/openverse-api/blob/a7955c86d43bff504e8d41454f68717d79dd3a44/api/catalog/api/models/models.py#L7-L19

@zackkrida
Copy link
Member

@WordPress/openverse-frontend We should remove these links while they're not working, as a hotfix for a production bug. I created a new issue to discuss next steps after that.

@zackkrida zackkrida added 🟥 priority: critical Must be addressed ASAP 🟧 priority: high Stalls work on the project or its dependents and removed 🟧 priority: high Stalls work on the project or its dependents labels Mar 22, 2022
@zackkrida zackkrida self-assigned this Mar 22, 2022
@krysal krysal removed the 🟧 priority: high Stalls work on the project or its dependents label Mar 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟥 priority: critical Must be addressed ASAP
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants