Update to React 16.9 and Hooks

[shanebdavis]

I went through all the components and updated them to React 16.9 using hooks. No more classes. Reduced code somewhat.

[dabonnestor]

You mean 16.9? not 12.9

[shanebdavis]

You mean 16.9? not 12.9

OMG yes! Thanks!

[shanebdavis]

I'd like to merge my work with pull-request #135 , but I'm going to hold off until it is accepted into the main repo.

[bluedusk]

If I'm not running it wrongly, this is breaking if you go to New Post it will keep firing:

 action EDITOR_PAGE_LOADED @ 13:47:14.096
redux-logger.js:1  prev state {article: {…}, articleList: {…}, auth: {…}, common: {…}, editor: {…}, …}
redux-logger.js:1  action     {type: "EDITOR_PAGE_LOADED", payload: null}
redux-logger.js:1  next state {article: {…}, articleList: {…}, auth: {…}, common: {…}, editor: {…}, …}
redux-logger.js:1  action EDITOR_PAGE_UNLOADED @ 13:47:14.102
redux-logger.js:1  prev state {article: {…}, articleList: {…}, auth: {…}, common: {…}, editor: {…}, …}
redux-logger.js:1  action     {type: "EDITOR_PAGE_UNLOADED"}

And the form is not editable.

[shanebdavis]

If I'm not running it wrongly, this is breaking if you go to New Post it will keep firing:
[...] And the form is not editable.

Thanks! I'll take a look.

[ghost]

In the Editor.js the second useEffect needs an empty array as the second argument.

useEffect(() => { if (props.match.params.slug) { return props.onLoad(agent.Articles.get(props.match.params.slug)); } props.onLoad(null); return () => props.onUnload(); }, []);

[ghost]

Also in the Editor.js
value={props.title || ''}
value={props.description || ''}
value={props.body || ''}
value={props.tagInput || ''}
|| '' was missing from the input.

[ghost]

In the Settings.js we have similar missing || '' on the inputs.
Looking good otherwise.