Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #1024 dispatched an action of resetCurrentMap to load metadatamodal #1221

Merged
merged 2 commits into from
Oct 27, 2016

Conversation

saidaipparla
Copy link
Contributor

after save action in user manager

@saidaipparla saidaipparla changed the title fix #1024 dipached an action of resetCurrentMap to load metadatamoda fix #1024 dispached an action of resetCurrentMap to load metadatamoda Oct 27, 2016
@saidaipparla saidaipparla changed the title fix #1024 dispached an action of resetCurrentMap to load metadatamoda fix #1024 dispatched an action of resetCurrentMap to load metadatamoda Oct 27, 2016
@saidaipparla saidaipparla changed the title fix #1024 dispatched an action of resetCurrentMap to load metadatamoda fix #1024 dispatched an action of resetCurrentMap to load metadatamodal Oct 27, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 79.756% when pulling fe426a3 on saidaipparla:permission_editor into 503c9b7 on geosolutions-it:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.007%) to 79.765% when pulling c8db190 on saidaipparla:permission_editor into 503c9b7 on geosolutions-it:master.

@mbarto mbarto merged commit 5a12fa1 into geosolutions-it:master Oct 27, 2016
case RESET_CURRENT_MAP: {
// resetting all the keys of the currentMap state
let newCurrentMap = Object.keys(state).reduce(function(previous, current) {
previous[current] = null;
return previous;
}, {});
return assign({}, newCurrentMap);
}*/
case RESET_CURRENT_MAP: {
return {};
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Please keep only the second RESET_CURRENT_MAP and remove the commented code

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.

4 participants