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

При удалении жалобы (тикета) удалять связь с нодой/комментарием #759

Closed
itcrowd72 opened this Issue Oct 30, 2017 · 9 comments

Comments

Projects
None yet
3 participants
@itcrowd72
Member

itcrowd72 commented Oct 30, 2017

При удалении жалобы (тикета) удалять связь с нодой/комментарием, чтобы не было "фантомных жалоб" как #663 и снова была возможность пожаловаться на эту сущность.

Готовлю PR

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 30, 2017

Contributor

@itcrowd72 так тот ISSUE и взял бы, зачем дублировать?
Переассайнить его на тебя?

Contributor

awd-studio commented Oct 30, 2017

@itcrowd72 так тот ISSUE и взял бы, зачем дублировать?
Переассайнить его на тебя?

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Oct 30, 2017

Member

Нет, тот issue должен будешь исправить ты :) Нужно изменить значение claim на 0 у ноды 2 в таблице node
Это нужно будет сделать сначала на деве, потом и на проде. А мой PR сделает так, чтобы таких фантомных жалоб не было в будущем при удалении тикетов

Member

itcrowd72 commented Oct 30, 2017

Нет, тот issue должен будешь исправить ты :) Нужно изменить значение claim на 0 у ноды 2 в таблице node
Это нужно будет сделать сначала на деве, потом и на проде. А мой PR сделает так, чтобы таких фантомных жалоб не было в будущем при удалении тикетов

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 30, 2017

Contributor

А, ну ок. Давай так ))

Contributor

awd-studio commented Oct 30, 2017

А, ну ок. Давай так ))

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 30, 2017

Contributor

По времени, за сегодня успеешь?

Contributor

awd-studio commented Oct 30, 2017

По времени, за сегодня успеешь?

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Oct 30, 2017

Member

С нодами уже разобрался, щас с комментами разберусь. Сегодня заморозка?

Member

itcrowd72 commented Oct 30, 2017

С нодами уже разобрался, щас с комментами разберусь. Сегодня заморозка?

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 30, 2017

Contributor

Да, я на какую веху направить нужно должен прикинуть.

Contributor

awd-studio commented Oct 30, 2017

Да, я на какую веху направить нужно должен прикинуть.

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 30, 2017

Contributor

А?

Contributor

mbaev commented Oct 30, 2017

А?

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 30, 2017

Contributor

Все что не успеваете сделать сегодня до 8ми мск, переносите на след спринт (веха)

Contributor

mbaev commented Oct 30, 2017

Все что не успеваете сделать сегодня до 8ми мск, переносите на след спринт (веха)

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Oct 30, 2017

Member

Готово, смотрите PR

Member

itcrowd72 commented Oct 30, 2017

Готово, смотрите PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment