Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Upgrade to react 16 - Closes #902 #938

Merged
merged 15 commits into from
Nov 2, 2017
Merged

Conversation

yasharAyari
Copy link
Contributor

Closes #902

@yasharAyari yasharAyari requested review from reyraa and slaweet October 31, 2017 22:50
@yasharAyari yasharAyari self-assigned this Oct 31, 2017
});

it('shows at least some progress on pressing input if this.isTouchDevice()', () => {
const wrapper = mount(<PassphraseGenerator {...props}/>);
const isTouchDeviceMock = mock(wrapper.instance()).expects('isTouchDevice');
isTouchDeviceMock.returns(true).twice();
wrapper.instance().setState({}); // to rerender the component
wrapper.find('.touch-fallback textarea').simulate('change', { target: { value: 'random key presses' } });
wrapper.update();
wrapper.find('Input.touch-fallback textarea').simulate('change', { target: { value: 'random key presses' } });
Copy link
Contributor

@reyraa reyraa Nov 1, 2017

Choose a reason for hiding this comment

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

I see you've added a tag name to the css selector in multiple lines including this, would you please explain the problem?

@reyraa reyraa changed the base branch from development to 1.3.0 November 1, 2017 09:32
@ginacontrino ginacontrino self-requested a review November 1, 2017 10:57
ginacontrino
ginacontrino previously approved these changes Nov 1, 2017
Copy link
Contributor

@ginacontrino ginacontrino left a comment

Choose a reason for hiding this comment

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

Thank you

Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

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

Well done @yasharAyari

@yasharAyari yasharAyari merged commit 773e746 into 1.3.0 Nov 2, 2017
@yasharAyari yasharAyari deleted the 902-upgrade-to-react-16 branch November 2, 2017 08:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants