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

add musicbrainz #681

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

qilinz
Copy link

@qilinz qilinz commented Jul 6, 2024

No description provided.

@alphatownsman
Copy link
Member

Hi, knowing this is still wip (and please take time as it suites you), just fyi some small changes in the internal scraping data structures, now your scraper will need to return title and description in a list of dict like [{'lang':'en,'text':'the title'}]; and ideally if the site you get data from supports more languages, you can either store all of them in the list, or store a selected few based on site configuration in common.models.lang.PREFERRED_LANGUAGES

@qilinz
Copy link
Author

qilinz commented Jul 18, 2024

Thank you! May i ask how can i run cat command now to download the website? @alphatownsman

@alphatownsman
Copy link
Member

Instead of python manage.py cat, now you use docker compose --profile dev run --rm shell neodb-manage cat

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Wishlist
Development

Successfully merging this pull request may close these issues.

2 participants