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

[DUPLICATE HAS-WORKAROUND] .cypher files missing after transpilation #62

Closed
anrolmar opened this issue Nov 4, 2020 · 3 comments
Closed
Labels
duplicate This issue or pull request already exists has-work-around

Comments

@anrolmar
Copy link

anrolmar commented Nov 4, 2020

After the build step, there are no .cypher files in the dist directory. That is because of tsc behaviour to only include ts/js files. How did you manage to get it to work?

Cannot find module '/****/api/dist/user/repository/userById.cypher'
Require stack:

/****/api/node_modules/@liberation-data/drivine/DrivineInjectionDecorators.js
@jasperblues
Copy link
Member

Please follow #34 @anrolmar. Meanwhile, here is a work-around.

Workaround

I will provide an official solution soon, in the meantime can you please write your queries inline (in the code)? For example:

const spec = new QuerySpecification(`MATCH (n) return (n)`)

Alternatively, you might modify your build/watch script to copy those files over. If using watch it could be a bit tricky, but might look something like:

Package.json:

"watch-dev": "rimraf dist && cross-env NODE_ENV=development ./node_modules/.bin/tsc-watch --onSuccess \"copyCpyherFiles.sh""

. . where copyCypherFiles.sh is an exercise for the reader at this point.

@jasperblues jasperblues changed the title .cypher files missing after transpilation [DUPLICATE] .cypher files missing after transpilation Nov 4, 2020
@jasperblues jasperblues added has-work-around duplicate This issue or pull request already exists labels Nov 4, 2020
@jasperblues jasperblues changed the title [DUPLICATE] .cypher files missing after transpilation [DUPLICATE HAS-WORKAROUND] .cypher files missing after transpilation Nov 4, 2020
@anrolmar
Copy link
Author

anrolmar commented Nov 4, 2020

Thank you very much. But I think it would be a great feature to include.

@jasperblues
Copy link
Member

Yes @anrolmar it is high priority. Features most asked for (not formally measured):

#34 : This issue.
#32 : Clear guides to use without Nest JS. (and NestJS pulled in as optional npm addon).
#36 : Don't pull in other platform's driver.

Currently working on RedisGraph Support, in between a lot of client work. Will happily take PRs, documentation or any other help large or small (including logging issues and sharing feedback 👍 ).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists has-work-around
Projects
None yet
Development

No branches or pull requests

2 participants