Víctor Fernández de Alba
Víctor Fernández de Alba
@davisagli yes, we were already doing this, it's on the generator since long ago: https://github.com/plone/volto/blob/master/packages/generator-volto/generators/addon/templates/babel.config.js but it's true that we were removing it in the docker implementations because we relied...
I made good progress using trying `process.chdir('src/addons/my-addon')` , then run the original script, still some rough edges... and we will have to get back the babel config.
@davisagli since it's something that will work even without the dockerized approach, I would include it anyways. In fact, the generator has it since some time ago, so it's already...
@Hrittik20 you can use: ``` pnpm test -- -u ``` It does the trick, by passing args to pnpm scripts (as it did always, it's not a pnpm trick). If...
@Gomez applied suggestions
@pbauer I pinned the p.restapi version to 9.7.0, however, it's still failing in a variety of places, some unrelated to the change :( I can take a look during the...
@davisagli @pbauer I've fixed the tests but one, the teaser itself. Now the docker image gets the right p.restapi version. I tried to fix it, but I realised that we...
Ok, I see it now: https://github.com/plone/plone.restapi/pull/1788/files#diff-1d97712ee7f4214036de21d1867ecf54f50381365e1efccdb84bfc5ea4df9c37R217 I think I can take care of it
@pbauer @davisagli I think now it's good to go. @plone/volto-team Can I have a quick review of one of you?
@tisto already taken care, if you start the docker container: `make start-backend-docker` then start the frontend, `pnpm start` in this PR you'll have it ready to test.