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

align geostore.properties to commit 9ba3bac9bea7dfd6d3c899ee93aeaf6a049f058a #194

Merged

Conversation

lpasquali
Copy link
Contributor

Hello @fvanderbiest

since we updated the name of the pg schema from "geostore" to "mapstore" we need also to modify this file to make mapstore awayre of the correct schema.
Also the schema name change affects two files in georchestra/mapstore2-georchestra repository:
https://github.com/georchestra/mapstore2-georchestra/tree/master/database

@fvanderbiest
Copy link
Member

fvanderbiest commented Sep 23, 2020

Yes.

Regarding the SQL files, their integration in the geOrchestra project is still pending, with the (ill-named) georchestra/georchestra#3058

@lpasquali
Copy link
Contributor Author

Yes.

Regarding the SQL files, their integration in the geOrchestra project is still pending, with the (ill-named) georchestra/georchestra#3058

the sigle sql file with mapstore name was merged short while ago: https://github.com/georchestra/mapstore2-georchestra/tree/master/database

@fvanderbiest
Copy link
Member

the sigle sql file with mapstore name was merged short while ago: https://github.com/georchestra/mapstore2-georchestra/tree/master/database

OK, but IMO it should also be added in https://github.com/georchestra/georchestra/tree/master/postgresql

@lpasquali
Copy link
Contributor Author

lpasquali commented Sep 23, 2020

conflicts resolved, this can be merged @fvanderbiest

@lpasquali lpasquali force-pushed the mapstore_integration branch 3 times, most recently from 9085b78 to 6c9d684 Compare September 24, 2020 10:02
@fvanderbiest fvanderbiest merged commit 1c19c6d into georchestra:docker-master Sep 25, 2020
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.

2 participants