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

Fix/graphql #181

Merged
merged 8 commits into from
Jan 9, 2025
Merged

Fix/graphql #181

merged 8 commits into from
Jan 9, 2025

Conversation

kirill-ivanovvv
Copy link
Contributor

add log to debug #178

image
судя по вот этому ^ полю ошибка в конкретном запросе. на него и повесил лог

@kirill-ivanovvv kirill-ivanovvv self-assigned this Jan 9, 2025
Copy link
Contributor

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

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

@kirill-ivanovvv
Copy link
Contributor Author

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

для каждого запроса в этой странице сделать по аналогии с этим?

@Nelfimov
Copy link
Contributor

Nelfimov commented Jan 9, 2025

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

для каждого запроса в этой странице сделать по аналогии с этим?

Нет - у тебя же в логах типовой объект ошибки apollo. Но в нем не видны объекты по понятным причинам. Надо это исправить

Copy link
Contributor

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

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

https://www.apollographql.com/docs/apollo-server/data/errors

@kirill-ivanovvv
Copy link
Contributor Author


if (graphQLErrors) {
graphQLErrors.forEach(({ message, locations, path, extensions }) => {
console.log(`[GraphQL error]: Message: ${message}`)
Copy link
Contributor

Choose a reason for hiding this comment

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

debug или error будет правильнее

blog/app/package.json Show resolved Hide resolved
@Nelfimov Nelfimov merged commit c19f38e into master Jan 9, 2025
5 of 6 checks passed
@Nelfimov Nelfimov deleted the fix/graphql branch January 9, 2025 11:15
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.

2 participants