search

RedMadRobot / navidux

10 stars 6 forks source link

add navigation finish point #9

Closed Bananzaaa closed 10 months ago

Bananzaaa commented 11 months ago

Add navigation finish flow point to navigation start place

darkkifir commented 10 months ago

У нас точно нет способа это сделать не отдельным колбеком, а в рамках существующего. Иной вариант пересмотреть это вообще)

Bananzaaa commented 10 months ago

У нас точно нет способа это сделать не отдельным колбеком, а в рамках существующего. Иной вариант пересмотреть это вообще)

ну в нынешних реалиях gotUpdatedData тесно завязан на ViewController, что делает поведение непоследовательным (но оно может быть удобно в случае навигации через VC). Callback в новой реализации, кмк, как раз делает именно то, что и предполагается, а именно возвращает сигнал в место вызова. gotUpdatedData в этом случае повторяет поведение делегата. Возможно, есть смысл задепрекейтить gotUpdatedData, так как по сути, замыкание можно юзать и на ViewController (но нужно проверить, что ничего не поломается, конечно)

darkkifir commented 10 months ago

У нас точно нет способа это сделать не отдельным колбеком, а в рамках существующего. Иной вариант пересмотреть это вообще)

ну в нынешних реалиях gotUpdatedData тесно завязан на ViewController, что делает поведение непоследовательным (но оно может быть удобно в случае навигации через VC). Callback в новой реализации, кмк, как раз делает именно то, что и предполагается, а именно возвращает сигнал в место вызова. gotUpdatedData в этом случае повторяет поведение делегата. Возможно, есть смысл задепрекейтить gotUpdatedData, так как по сути, замыкание можно юзать и на ViewController (но нужно проверить, что ничего не поломается, конечно)

Тогда может так и надо сделать?) Либо задепрекейтить, либо просто эту же логику перенести на gotUpdatedData