These are chat archives for LaravelRUS/chat

7th
Mar 2018
projct1
@projct1
Mar 07 2018 10:21
подскажите плз по гиту:
если чувак сделал пул реквест
я ему там в комментах написал что где поправить
надо отклонять пул реквест, чтобы он всё поправил и создал новый?
или можно как-нибудь по-другому, то есть внести ему изменения в текущий пул-реквест, согласно комментам?
Vlad Shcherbin
@VladShcherbin
Mar 07 2018 12:18
@projct1 у него есть ветка этого пул реквеста, когда он в неё делает коммиты, его пул реквест обновляется, отклонять не нужно
projct1
@projct1
Mar 07 2018 13:22
@VladShcherbin она автоматогм создаётся когда пул реквест создаёт он?
Vlad Shcherbin
@VladShcherbin
Mar 07 2018 13:22
@projct1 да
projct1
@projct1
Mar 07 2018 13:34
@VladShcherbin то есть пушить из phpstorm надо будет уже не в мастер а в новую ветку?
Vlad Shcherbin
@VladShcherbin
Mar 07 2018 13:34
@projct1 да
projct1
@projct1
Mar 07 2018 13:42
@VladShcherbin gа потом вручную сливать эту ветку с мастером?
Vlad Shcherbin
@VladShcherbin
Mar 07 2018 13:43
@projct1 примешь пул реквест и автоматически добавится в мастер
projct1
@projct1
Mar 07 2018 13:43
@VladShcherbin круто, спс
@VladShcherbin спс
убрали чтоль карму?
Vlad Shcherbin
@VladShcherbin
Mar 07 2018 14:50
не, бот как обычно отвалился
Tarasovych
@Tarasovych
Mar 07 2018 15:30
Странный вопрос, но вдруг:
где здесь \vendor\laravel\framework\src\Illuminate\Foundation\Auth\ResetsPasswords.php
происходит валидация токена? именно на соответствие его с БД (required в rules() - это отдельная тема)
метод, который срабатывает в случае невалидного токена, нашел - sendResetFailedResponse()
JhaoDa
@jhaoda
Mar 07 2018 16:18
@Tarasovych ты не поверишь, но в методе reset