Skip to content

#3237 Removes the use of StandardRouter from MapStore 2 jsapi#3240

Merged
afabiani merged 1 commit intogeosolutions-it:geonode_integrationfrom
kappu72:geonode_integration
Oct 15, 2018
Merged

#3237 Removes the use of StandardRouter from MapStore 2 jsapi#3240
afabiani merged 1 commit intogeosolutions-it:geonode_integrationfrom
kappu72:geonode_integration

Conversation

@kappu72
Copy link
Copy Markdown
Contributor

@kappu72 kappu72 commented Oct 15, 2018

Description

StandardRouter is removed in favour of StandardContainer.

Issues

Please check if the PR fulfills these requirements

@ghost ghost assigned kappu72 Oct 15, 2018
@afabiani afabiani merged commit f09785f into geosolutions-it:geonode_integration Oct 15, 2018
@mbarto
Copy link
Copy Markdown
Contributor

mbarto commented Oct 16, 2018

Can we make this improvement to MS2 master with a little bit of refactoring to get common stuff not replicated between StandardRouter and StandardContainer?

@kappu72
Copy link
Copy Markdown
Contributor Author

kappu72 commented Oct 16, 2018

Can we make this improvement to MS2 master with a little bit of refactoring to get common stuff not replicated between StandardRouter and StandardContainer?
Yes, I guess to remove all common stuff from StandardRouter and wrap this in a StandardContainer.
any objection?

@mbarto
Copy link
Copy Markdown
Contributor

mbarto commented Oct 16, 2018

@kappu72 looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants