Skip to content

Homework 4#45

Open
galusha wants to merge 3 commits intoromabelka:masterfrom
galusha:Homework---4
Open

Homework 4#45
galusha wants to merge 3 commits intoromabelka:masterfrom
galusha:Homework---4

Conversation

@galusha
Copy link
Copy Markdown

@galusha galusha commented Dec 2, 2018

No description provided.

static propTypes = {}

componentWillUpdate() {
this.refs.forceUpdateGrid()
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.

Очень странно, что тебе это нужно

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Более того, убила на это пару часов. Мне хотелось, чтобы при добавлении в selectedEventCard человека, в selectedEventCard добавленные люди отображались бы списком. Но они не отображались! Т.е. selectedEventCard перерендеривался, а вот List в нем - нет, так как количество выбранных events не менялось, поэтому пришлось рендерить его вручную

Comment thread src/ducks/events.js
const lastEvent = state.entities.last()

const data = yield call(api.fetchLazyEvents, lastEvent && lastEvent.id)
const data = yield call(api.fetchLazyEvents, lastEvent && lastEvent.get('id'))
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.

Если у тебя lastEvent Record - .get('id') лишнее

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