Skip to content

Homework 5#54

Open
galusha wants to merge 2 commits intoromabelka:masterfrom
galusha:Homework-5
Open

Homework 5#54
galusha wants to merge 2 commits intoromabelka:masterfrom
galusha:Homework-5

Conversation

@galusha
Copy link
Copy Markdown

@galusha galusha commented Dec 5, 2018

No description provided.

import { connect } from 'react-redux'
import { addPersonToEvent } from '../../ducks/events'
import { peopleByIdsSelector } from '../../ducks/people'
import { Motion, spring } from 'react-motion'
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Так ты не сможешь удаление анимировать, по сути, этот компонент умрет вместе с тем, который он должен был бы анимировать

Comment thread src/ducks/auth.js
const chanel = yield call(createAuthStateChanel)

while (true) {
const { data } = yield take(chanel)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Можно еще, если нет user, SIGN_OUT делать

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