Closed LarryEitel closed 9 years ago
nice catch WebStorm is so strong, I will keep a look, may be caused by ES6 new features http://www.2ality.com/2014/09/es6-modules-final.html but normally just a warning
I figured that much. I am new at ES6 as well. I'm using your project to learn. :) I looked at many but yours includes the key pieces I'm looking for. Especially do I appreciate your $ajax example.
nice, me too I have learned by creating it, inspect, play, redo, good for both of us, you will build website quickly, enjoy Larry. do not hesitate
Changing in AppActions
module.exports = alt.createActions(AppActions);
To:
export default alt.createActions(AppActions);
Makes WebStorm happier and doesn't seem to break anything.
Thoughts?
yes it is but I will add it later, thx Larry, or do not hesitate put a PR for it
I am finding more places where same refactoring is required. Will submit a PR next chance.
:+1: sure a big refactoring is needed and it is difficult to make it alone :)
Happy to help where/when I can. Are you open to discuss via Hangout/Skype?
On Tue, Aug 11, 2015 at 9:33 AM, Julien Valéry notifications@github.com wrote:
[image: :+1:] sure a big refactoring is needed and it is difficult to make it alone :)
— Reply to this email directly or view it on GitHub https://github.com/darul75/web-react/issues/14#issuecomment-129931638.
I would like so much but it is blocked by our company proxy there ;) otherwise on darul75@gmail.com when at home
Hello @darul75, been MIA on this project because I have been focusing on Redux
vs Alt
.
Same issue with PHPStorm, messing things up here and then. Most of the times, it works and then suddenly stops working. Solution is to duplicate it and change the name, and it starts working again. Seriously, a mess.
I'm using it for multiple exports as
module.exports = {
...
...
}
'...\app\actions\AppActions.js' line 5:
import AppActions from '../../actions/AppActions';
WebStorm complains/suggests that
AppActions
Default export is not declared in imported module.
Just a curious message.