Skip to content

Conversation

@haailabs
Copy link

@haailabs haailabs commented Nov 30, 2023

Changed NEXT_PUBLIC_METADATACACHE_URI from 'https://aquarius.waves.oceanprotocol.com' to 'https://v4.aquarius.oceanprotocol.com'

Changes proposed in this PR:
The API at https://aquarius.waves.oceanprotocol.com fails to retrieve metadata for new submissions and existing ones. I suggest using v4.aquarius.oceanprotocol.com instead.

More details:
The showcase dapp at https://waves.oceanprotocol.com/ currently fails to display new submissions (tested on Mumbai) and the API call returns "not found in Elasticsearch". Clicking on "all audio files" in https://waves.oceanprotocol.com/ only displays four results. Using v4.aquarius.oceanprotocol.com, displays 139 results, and the current user's new submissions.

waves
v4

@netlify
Copy link

netlify bot commented Nov 30, 2023

👷 Deploy request for spiffy-cendol-e83911 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 1b6ebb9

The function is called create() not createAlbum()
Added a section on Changing Asset Types
@haailabs
Copy link
Author

haailabs commented Dec 4, 2023

Corrected createAlbum(), not currently used in the code, to create(), in the docs.
Added a section on how to change the asset type (e.g. to mp4 movies) for submissions and indexing.

@LoznianuAnamaria
Copy link
Contributor

LoznianuAnamaria commented Dec 4, 2023

@haailabs Thanks for opening this PR 🙏 Your contributions are highly valued.
@mihaisc let's review it please

@mihaisc
Copy link
Collaborator

mihaisc commented Dec 4, 2023

Hi @haailabs thanks for the interest and effort. The aquarius url is correct and working as expected. This is also a demo of using aquarius as an "isolated marketplace" where it indexes only approved publishers, in this case only the Jelly McJellyfish account. It was never meant to be a product. If you want to expand on it you are free to use the codebase and deploy it, we are glad to help

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