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

Mag #24

Merged
merged 18 commits into from
Oct 2, 2023
Merged

Mag #24

merged 18 commits into from
Oct 2, 2023

Conversation

MichaeL-JSON
Copy link
Owner

No description provided.

Создан контейнер БД.
Настроено использование в NestJS абсолютных путей к файлам.
Настроено подключение NestJS к БД.
Добавлены скрипты для работы с TypeORM.
Сгенерирована миграция для создания таблицы articles.
Настроено получение статей стороннего API.
добавлено ограничение уникальности на столбец title таблицы articles,
добавлен столбец tickers в таблицу articles.
… строк.

Теперь в БД добавляются только те данные, которых ещё нет в базе.
Попытка сохранения дублирующихся строк в БД не вызывает остановку сервера.
@MAG1980 MAG1980 self-requested a review September 23, 2023 18:04
т.к. при запуске NestJS в контейнере Docker не работает 'hot reload'.
Эндпойнт для первоначального заполнения БД изменён на /articles/fetch.
Создан эндпойнт /articles, возвращающий массив из всех статей, хранящихся в БД.
Установлен пакет bcrypt для хеширования паролей.
Создан эндпойнт /users (POST), предназначенный для регистрации пользователей.
@MAG1980
Copy link
Collaborator

MAG1980 commented Oct 2, 2023

На порту 5000 с помощью Docker Compose развёрнут REST API на основе NestJS, TypeORM и Postgres.
Настроен регулярный (в начале пятой минуты каждого часа) парсинг статей со стороннего ресурса для сохранения их в локальной БД.
Созданы эндпойнты:
получения массива всех статей: (GET) localhost:5000/articles;
получение конкретной статьи по id: (GET) localhost:5000/articles/:id;
создание пользователя: (POST) localhost:5000/users.

@MAG1980 MAG1980 closed this Oct 2, 2023
@MAG1980 MAG1980 reopened this Oct 2, 2023
@MAG1980 MAG1980 merged commit f8c598e into main Oct 2, 2023
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.

None yet

2 participants