-
Notifications
You must be signed in to change notification settings - Fork 1
Experiences while setting up a new project via https://github.com/OnroerendErfgoed/atramhasis_demo_cookiecutter#readme #1
Comments
I also ran into the pyramid_openapi3 issue. Seem like that should be installed by atramhasis. I did not need to install jinja though. |
Made OnroerendErfgoed/atramhasis#697 for the openapi3 issue |
What version of jinja2 do you have installed? When installing all packages using
A
Manually installing an older version of jinja fixes the exception for me.
I guess the latest version of pryamid_jinja2 is not compatible with the latest version of jinja2 (since jinja 3.1.0: https://github.com/pallets/jinja/pull/1544/files). They are installed by including atramhasis where both libraries are not pinned at a specific version in the setup.py file: https://github.com/OnroerendErfgoed/atramhasis/blob/f4dba0e37be1ebe50bc1dc2a1989e56c559d182c/setup.py#L57-L58. At this moment I'm not sure where the versions should be pinned. |
The jinja2 problem is actualy this line: https://github.com/OnroerendErfgoed/atramhasis_demo_cookiecutter/blob/main/%7B%7Bcookiecutter.repo%7D%7D/development.ini#L22 I have created issues #3 and OnroerendErfgoed/atramhasis_scaffold_cookiecutter#1 to fix this in the impacted atramhasis repositories |
@roefem: The only things from the description above that are not cleared out are:
|
There is no admin application typescript/javascript present in the demo admin folder, so installing those dependencies is not needed. Maybe we can omit the admin folder from this cookiecutter project?
To reproduce this you'll need to make sure you are using a more recent Node version. But as you say, it's not an issue. |
It is confusing to me to have this folder with a package.json file when it has no use. I prefer to drop it. I will create issues in atramhasis_demo_cookiecutter and atramhasis_scaffold_cookiecutter |
Don't we need an npm install of the static/admin folder?
qucik fix
pip install pyramid_openapi3==0.13
Stacktrace:
quick fix:
pip install jinja2==3.0.3
The text was updated successfully, but these errors were encountered: