-
Notifications
You must be signed in to change notification settings - Fork 4
[bug] [content-publishing] "class-validator" has no exported member named 'ValidationOptions' #798
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
Comments
Are you running this from the
Also, verify which image you're running:
(Either way, both published & dev images work for me.) But--maybe a platform issue? I'm on Docker Desktop for macOS. (Though I'd think this error would not be affected by the host or build platform, since it's a Node.js runtime issue). However:
I tested:
All of the above work for me. I haven't booted my Linux box in a while, but I'll see if I can give it a try later. Other that that, I'd say not reproducible for me. |
Now that I look at your error again, it looks more likely that you ran the Again, the only difference would be the platform, but I don't think a platform issue would manifest this type of TS/JS error. |
I encountered an error when running the Gateway repo related to the Content Publishing API. After investigating, I found that the file app/node_modules/class-validator/types/decorator/ValidationOptions.d.ts was empty in the Content Publishing Server Docker container. To resolve the issue, I filled in the file with the contents from the ValidationOptions.d.ts of the class-validator module, and it worked successfully. |
Was this a dev container that you built locally, or the published container?
My money's on a hiccup (probably network-related, during the 'npm ci' process) in the container build process. However, maybe the image build script should validate that the |
It was a dev container |
Still unclear as to whether you're running this from the SAT repo or the Gateway repo, as "dev container" means 2 different things in those 2 contexts. If you're running from the SAT repo, I'd suggest rebuilding your dev images from this (Gateway) repo using If you're running from this repo, "dev" containers map to the local filesystem for the app source, but store their |
When I run the Gateway repo with Dev Containers, I encounter an error in the Content Publishing API.
The text was updated successfully, but these errors were encountered: