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

Add cadastrapp #185

Closed
wants to merge 1 commit into from
Closed

Add cadastrapp #185

wants to merge 1 commit into from

Conversation

RemiDesgrange
Copy link
Contributor

@RemiDesgrange RemiDesgrange commented May 4, 2020

(PR on georchestra/docker in progress here : georchestra/docker#57)

@RemiDesgrange RemiDesgrange changed the base branch from master to docker-master May 4, 2020 13:22
@RemiDesgrange RemiDesgrange marked this pull request as draft May 4, 2020 13:58
@RemiDesgrange RemiDesgrange self-assigned this May 4, 2020
@RemiDesgrange RemiDesgrange marked this pull request as ready for review May 15, 2020 12:06
@landryb
Copy link
Member

landryb commented May 20, 2020

dunno how branch merges are handled in that repo, but if this is merged, would be nice to have in master too - for ppl deploying outside docker :) Fwiw, the file was removed by @fvanderbiest in b947651 (i dont remember the reason) ..

@fvanderbiest
Copy link
Member

would be nice to have in master too

Indeed, this PR should target master, and then master should be merged into docker-master.

Fwiw, the file was removed by @fvanderbiest in b947651 (i dont remember the reason) ..

Yeah, changed my mind. Easier to manage here, even if the module is not part of the core...

# PDF generation
pdf.dateValiditeDonneesMajic=01/01/2017
pdf.dateValiditeDonneesEDIGEO=01/06/2017
pdf.organisme=Georchestra PSC
Copy link
Member

Choose a reason for hiding this comment

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

geOrchestra

# only used when wms service need authentification
# if empty no authentification is used
cadastre.wms.username=geoserver_privileged_user
cadastre.wms.password=geoserver
Copy link
Member

Choose a reason for hiding this comment

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

This is not the datadir's default password for geoserver_privileged_user

parcelle.wms.password=

cadastre.wfs.url=https://georchestra.mydomain.org/geoserver/wfs
cadastre.wfs.layer.name=pci:parcelle_2020
Copy link
Member

Choose a reason for hiding this comment

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

Does this mean we should also provide this layer by default with https://github.com/georchestra/geoserver_minimal_datadir ?

Copy link
Member

Choose a reason for hiding this comment

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

especially since pci is available on https://cadastre.data.gouv.fr/data/dgfip-pci-vecteur/ ... but that means having a script to bootstrap the gs datadir fetching data from there, not going to import pci data in the repo :)


## Temp folder with write acces to create image and temporary files needed for pdf generation
## This folder should be writtable by tomcat user
tempFolder=/tmp
Copy link
Member

Choose a reason for hiding this comment

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

Not docker compatible I guess ...

@edevosc2c
Copy link
Member

Cadastrapp is not part of georchestra anymore.

@edevosc2c edevosc2c closed this Jun 13, 2023
@edevosc2c edevosc2c deleted the add-cadastrapp branch September 19, 2023 09:56
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