Skip to content

Покоритель баз данных - #5

Merged
keksobot merged 2 commits into
htmlacademy-nodejs-api:masterfrom
Critonik:module4-task1
Jan 28, 2026
Merged

keksobot merged 2 commits into
htmlacademy-nodejs-api:masterfrom
Critonik:module4-task1

Conversation

@Critonik

@Critonik Critonik commented Jan 24, 2026 •

Copy link
Copy Markdown

@keksobot keksobot changed the title Подключил монгу Покоритель баз данных Jan 24, 2026
Comment thread src/cli/commands/command.constant.ts Outdated
@@ -0,0 +1,2 @@
export const DEFAULT_DB_PORT = '27017';
export const DEFAULT_USER_PASSWORD = '123456';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

не не не, такое в git нельзя выкладывать такое хранят в .env и не выкладывают в гит.
Даже не смотря на что это дефотлные штуки.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

выпилил

Comment thread package.json Outdated
"mock:server": "json-server ./mocks/mock-server-data.json --port 3123"
},
"dependencies": {
"@typegoose/typegoose": "^13.1.0",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

зависимости лучше ставить с фиксированной версией пакета без ^ чтобы в случае когда выйдут допустим какие то новые версии спустя пару лет у тебя не получилось так, что ты делаешь npm i и у тебя ничего запускается из за разницы в версиях пакета

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

поправил пакеты

Comment thread src/cli/commands/import.command.ts Outdated
avatar: offer.author,
userType: UserType.COMMON,
name: 'generate name',
email: 'email',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@prop({ unique: true, required: true })
public email: string;

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

поправил

@keksobot
keksobot merged commit e4a8222 into htmlacademy-nodejs-api:master Jan 28, 2026
1 check passed
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