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.

Sign up for GitHub

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

Jump to bottom

Moved the pdf from gdrive to repository. Closes #1644 #1645

Merged
merged 5 commits into from
Nov 8, 2024
Merged

Moved the pdf from gdrive to repository. Closes #1644 #1645

merged 5 commits into from
Nov 8, 2024

Conversation

artengin
Copy link
Contributor

@artengin artengin commented Nov 5, 2024

Issues fixed #1644

Moved the SICP books from google drive to the repository.

fey
fey reviewed Nov 7, 2024
View reviewed changes
resources/views/layouts/_nav.blade.php Outdated
@@ -29,7 +29,7 @@ class="nav-link p-2">{{ __('layout.nav.sicp_read') }}</a></li>
@endif
<li class="nav-item"><a href="{{ route('top.index') }}" class="nav-link p-2">{{ __('layout.nav.rating') }}</a>
</li>
<li class="nav-item"><a href="{{ TemplateHelper::getBookLink($currentLocale) }}"
<li class="nav-item"><a href="{{ Vite::asset("resources/assets/images/sicp-{$currentLocale}.pdf") }}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

@artengin тк это не images (картинки), то файлик лучше подвинуть в другое место, например в resources/pdf

fey
fey reviewed Nov 7, 2024
View reviewed changes
resources/views/layouts/_nav.blade.php
@@ -29,7 +29,7 @@ class="nav-link p-2">{{ __('layout.nav.sicp_read') }}</a></li>
@endif
<li class="nav-item"><a href="{{ route('top.index') }}" class="nav-link p-2">{{ __('layout.nav.rating') }}</a>
</li>
<li class="nav-item"><a href="{{ TemplateHelper::getBookLink($currentLocale) }}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

И возможно хелпер можно удалить, если он больше нигде не используется.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

у хелпера еще метод getTitleContent используется, остальное подчистил.
Добавил в ссылки target="_blank" думаю так будет удобнее пользователю

@fey fey merged commit 9b88dfe into Hexlet:main Nov 8, 2024
2 checks passed
@fey fey mentioned this pull request Nov 28, 2024
Merged
@fey
Copy link
Collaborator

fey commented Nov 28, 2024

@artengin Добрый день.

Откатил изменения, почему-то сервис падал. МОжете пожалуйста, проверить локально + задеплоить демку? Коммит с ревертом #1646

@artengin
Copy link
Contributor Author

@fey Добрый день.
Проверил, локально все работает, но я не устанавливал БД.
Код, который я менял, вроде никак не должен сказаться на проекте в целом.

Единственный момент, после того как я добавил в import.meta.glob строчку про /assets/pdf/, я выплнил команду
npm run build и все заработало на локалке

"задеплоить демку" можно подробнее?

@fey
Copy link
Collaborator

fey commented Nov 28, 2024

@artengin задеплоить приложение на render.com чтобы можно было потыкать кнопочки)

@artengin
Copy link
Contributor Author

К сожалению никак не могу задеплоить, выдает ошибку:

could not translate host name "database" to address: Name or service not known
make: *** [Makefile:33: db-prepare] Error 1

Вписывал в .env, то что указано в README:

DB_CONNECTION=pgsql
DB_HOST=database
DB_PORT=5432
DB_DATABASE=postgres
DB_USERNAME=postgres
DB_PASSWORD=secret

Нужна помощь)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fey fey fey left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Move russian hexlet sicp pdfs from gdrive to repository
2 participants
@artengin @fey