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

Created popularity field for Readmanga #106

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

NikDark
Copy link
Collaborator

@NikDark NikDark commented Aug 25, 2021

Описание

Добавил field popularity в модель манги
Добавил чтобы при парсинге листа из Readmanga запоминалась позиция


closes #92

@NikDark NikDark changed the title [WIP] Created popularity field for Readmanga Created popularity field for Readmanga Aug 31, 2021
@NikDark NikDark requested a review from dhvcc August 31, 2021 12:18
Copy link
Member

@dhvcc dhvcc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Всё круто, осталось только понять, что делаем по поводу headless

@@ -34,7 +34,7 @@ async def get_list(self):
self.logger.info("Starting list parser")
self.logger.info("=====================")
self.browser = await launch(
{"headless": True, "args": ["--no-sandbox", "--disable-setuid-sandbox"]}
{"headless": False, "args": ["--no-sandbox", "--disable-setuid-sandbox"]}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Навряд ли это запустится на EC2, там нет монитора

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NikDark решение есть?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нет, пока реально ничего не нашел в чем может быть проблема

@dhvcc dhvcc force-pushed the develop branch 2 times, most recently from 2123d47 to 6e70eb9 Compare October 19, 2021 18:13
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.

Запоминание позиции в лист вью
2 participants