-
Notifications
You must be signed in to change notification settings - Fork 9
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
[BUG] Generated index.ts files have syntax errors "property '_...Aspect' does not exist" #270
Comments
Are there any comments on my problem? |
Hi Stefan, sorry for the wait, I was OOO. I will look into it. Best, |
I have prepared a fix for this. Would you like to check if it works for your model before I release it? |
I would like to do so, but I am not sure what I have to do in order to link our repo to the feature branch of your repo. I assume this is possible, but I have never done this. If you tell me what I have to do I can give it a try. |
Sorry, I should have led with that! Please find this guide attached. If you need any further assistance in setting it up, just let me know. |
Could you please add the error message you are seeing there? |
Hi Daniel I just re-generated the files in @cds-models with the local link to your feature branch. I checked now all generated files in @cds-models for further syntax errors. Unfortunately, I found the following additional ones: The error
occurs in the files
Several additional errors, like e.g.
occur in the file
I have pushed these in the branch switch-to-cds-typer-payment-master-data (see https://github.tools.sap/erp4sme/crypto-for-business/tree/switch-to-cds-typer-payment-master-data/%40cds-models) Shall I open a separate bug for these, or can you check further in within this bug? Thank you, |
Hi Stefan, guess there was some caching or stale files going on, no worries. 🙂 Best, |
I created #278 as follow-up for the new issues. |
Is there an existing issue for this?
Nature of Your Project
TypeScript
Current Behavior
In our project we have defined the aspect SourceReferences, which is used in several entities.
The generated index.ts files containing these entities have al the syntax error
Somehow the index.ts files for the aspect seems not to be correctly generated.
Expected Behavior
The generated index.ts files shall be syntax error free.
Steps To Reproduce
repo https://github.tools.sap/erp4sme/crypto-for-business
npm run cds-typer:dev generates the files into folder @cap-models
Note: this script makes use of --IEEE754Compatible true in order to get cds type Decimal generated as (number | string)
Environment
Repository Containing a Minimal Reproducible Example
https://github.tools.sap/erp4sme/crypto-for-business
branch switch-to-cds-typer-payment-master-data
aspect SourceReferences is defined here: db/masterData/common/SourceReferences.cds
Anything else?
No response
The text was updated successfully, but these errors were encountered: