Skip to content

Сервер наносит ответный удар - #8

Merged
keksobot merged 2 commits into
htmlacademy-nodejs-api:masterfrom
MikhailBespyatov:module6-task1
Nov 13, 2025
Merged

keksobot merged 2 commits into
htmlacademy-nodejs-api:masterfrom
MikhailBespyatov:module6-task1

Conversation

@MikhailBespyatov

@MikhailBespyatov MikhailBespyatov commented Nov 12, 2025 •

Copy link
Copy Markdown

@keksobot keksobot changed the title module6-task1 Сервер наносит ответный удар Nov 12, 2025
this.addRoute({path: OfferPaths.Offer, method: HttpMethod.Delete, handler: this.deleteOffer});
}

public async index({query}: GetOffersRequest, res: Response): Promise<void> {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

не все требуемые поля есть в ответе.

5.4.6. Список возвращаемых полей предложения: стоимость аренды, название, тип жилья, флаг, позволяющий определить принадлежность к избранным предложениям, дата публикации, город, превью изображения, флаг «Премиум», рейтинг, количество комментариев.

this.ok(res, fillDTO(OfferDetailsRdo, existOffer));
}

public async updateOffer({body, headers, params}: UpdateOfferRequest, res: Response) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

this.ok(res, fillDTO(OfferDetailsRdo, result));
}

public async deleteOffer({params, headers}: Request, res: Response) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

throw OFFER_NOT_FOUND_ERROR;
}

this.ok(res, fillDTO(OfferDetailsRdo, existOffer));

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

отдается не полная информация. нет

  • количетво комментариев
  • рейтинг
  • isFavorite отдается неверный

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

не используется, удалить

throw USER_NOT_AUTHORIZED_ERROR;
}

const result = await this.offerService.findFavoriteOffersForUser('68ebc8354a76a93bd861bc45');

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

не хардкодить пользователя. пока брать его из dto

}

await this.favoriteService.toggleFavorite({
userId: '68ebc8354a76a93bd861bc45',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

не хардкодить пользователя. пока брать его из dto

@keksobot
keksobot merged commit 81b8c6c into htmlacademy-nodejs-api:master Nov 13, 2025
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