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

feat: append uniq validator for name field of content-type #40

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

lastdoctor
Copy link
Collaborator

No description provided.

@lastdoctor lastdoctor requested a review from EndyKaufman January 8, 2023 09:12
@lastdoctor lastdoctor linked an issue Jan 21, 2023 that may be closed by this pull request
private readonly prismaClintService: PrismaClientService;
constructor() {
DbService.instance = this;
this.prismaClintService = PrismaClientService.instance;
Copy link
Member

Choose a reason for hiding this comment

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

короче убери призму, не нужно делать универсальный способ, у тя по задаче идет конкретная логика для нее сделай

прикол с докидыванием реализации для работы с базой возьми тут https://github.com/EndyKaufman/kaufman-bot/blob/develop/libs/bot-in-groups/server/src/lib/bot-in-groups-config/bot-in-groups.config.ts

тайп ди выпиливай, я тебе как пример сказал что былобы круто его взять, но так как ты на неста идешь и тут мы решаем проблемы диай неста и модулей, то как бы тупо - брать левый диай, когда есть свой диай, кривой но есть

Copy link
Member

Choose a reason for hiding this comment

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

для общего понимания диай, юзать тайп ди, да это круто, я увидел что ты можешь и ты сам понял как юзать диай вне неста) цель достигнута, можно выпиливать тайп ди и брать нест диай

entity: string,
field: string
): Promise<boolean> {
const data = await this.prismaClintService[entity].findFirst({
Copy link
Member

Choose a reason for hiding this comment

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

убери универсальность, нужно четка по фиче

@@ -0,0 +1,52 @@
import {
Copy link
Member

Choose a reason for hiding this comment

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

вообще не нужно создать такой папки шаред, пуляй в папку фичи

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.

feat: append uniq validator for name field of content-type
2 participants